Merge ~jugmac00/lpci:fix-readthedocs into lpci:main
Proposed by
Jürgen Gmach
| Status: | Merged |
|---|---|
| Approved by: | Jürgen Gmach |
| Approved revision: | 7896aa0c138eb305084b58a6c1f2208f0836dbcd |
| Merge reported by: | Jürgen Gmach |
| Merged at revision: | 7896aa0c138eb305084b58a6c1f2208f0836dbcd |
| Proposed branch: | ~jugmac00/lpci:fix-readthedocs |
| Merge into: | lpci:main |
| Diff against target: |
39 lines (+9/-4) 2 files modified
.readthedocs.yaml (+4/-0) docs/conf.py (+5/-4) |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Colin Watson (community) | Approve | ||
|
Review via email:
|
|||
Commit message
Fix autodoc on readthedocs
Description of the change
The standard installation on readthedocs (`python setup.py install`)
does not work for this project, as we need some dependencies from e.g.
github which are only listed in requirements.txt
This is a best effort - I cannot verify it in advance, as rtd does not pick up my branch so I could render the docs :-/
To post a comment you must log in.
Revision history for this message
| Jürgen Gmach (jugmac00) wrote | # |
Revision history for this message
| Jürgen Gmach (jugmac00) wrote | # |
Revision history for this message
| Colin Watson (cjwatson) | # |
review:
Approve
There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
On the other hand - this won't be enough to fix the problem, as with `pip install -r requirements` lpcraft itself won't get installed.