-
Notifications
You must be signed in to change notification settings - Fork 276
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
DEP: move dev-only dependencies to requirements files #4983
DEP: move dev-only dependencies to requirements files #4983
Conversation
469e2e0
to
c4d8040
Compare
c4d8040
to
97a5fbf
Compare
Should update the instructions on building the docs too. This line:
|
97a5fbf
to
435e624
Compare
Good catch. Done ! |
oh, that did make the docs build fail though... |
Yes that's expected since the installation script used there is out of my control. |
@yt-fido test this please. |
435e624
to
621c71a
Compare
621c71a
to
5ad93c4
Compare
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
but one other docs note: https://yt-project.org/docs/dev/developing/testing.html#handling-yt-dependencies was out of date even before this PR :) if you feel like editing here that'd be great but I'm also OK with just opening an issue to track the need to update that page.
Thanks for pointing it out. Since the |
btw thanks @Xarthisius for your work on Jenkins, I'm glad this is working ! |
PR Summary
Close #4976
TODO: