-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MAINT: update rapids-dependency-file-generator to 1.17.1, fix docs builds #14
Conversation
Signed-off-by: James Lamb <jlamb@nvidia.com>
Signed-off-by: James Lamb <jlamb@nvidia.com>
af7e016
to
b39b96d
Compare
../../CONTRIBUTING.md |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I saw 2 warnings treated as errors in the docs builds:
/__w/legate-raft/legate-raft/docs/source/CONTRIBUTING.md:25: WARNING: Unknown source document '/__w/legate-raft/legate-raft/README' [myst.xref_missing]
/__w/legate-raft/legate-raft/docs/source/README.md:49: WARNING: Unknown source document '/__w/legate-raft/legate-raft/CONTRIBTING' [myst.xref_missing]
Renaming CONTRIBTING.md
to CONTRIBUTING.md
should fix one of them (the reference in docs/source/index.rst
was inconsistent with the actual file name), and maybe both?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ahhhh ok, so I still think this renaming (to fix the spelling error) was a good thing, but it did not resolve the failure.
Interesting to note that the docs this is complaining about are totally working in the rendered docs. You can go to https://rapidsai.github.io/legate-raft/README.html and try clicking "our guide for contributing to legate-raft" ... it leads to https://rapidsai.github.io/legate-raft/CONTRIBUTING.html successfully.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh! @seberg I just saw your comment here: rapidsai/legate-boost#212 (comment)
I guess it's sphinx
8.2.0 that is causing these warnings? I'll try pinning here too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right, that was one of the things I saw in legate-df.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah got it, ok! I've pushed a sphinx<8.2.0
pin here, it looks like it'd fix docs builds: https://github.com/rapidsai/legate-raft/actions/runs/13502337421/job/37726967372?pr=14
Signed-off-by: James Lamb <jlamb@nvidia.com>
3edd10d
to
f905f1d
Compare
Signed-off-by: James Lamb <jlamb@nvidia.com>
Signed-off-by: James Lamb <jlamb@nvidia.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, I checked that the modified links work in the artifact.
Contributes to rapidsai/build-planning#155
Miscellaneous small changes:
rapids-dependency-file-generator
(1.17.1)sphinx
to<8.2.0
(MAINT: update rapids-dependency-file-generator to 1.17.1, fix docs builds #14 (comment))CONTRIBTING.md
toCONTRIBUTING.md
(fixing typo)