-
Notifications
You must be signed in to change notification settings - Fork 2
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
DM-35680: Implement cache busting for RubinTV #89
Conversation
836537f
to
0dde1ec
Compare
requirements/dev.in
Outdated
|
||
-c main.txt | ||
|
||
aiohttp-devtools | ||
coverage[toml] | ||
google-cloud-storage | ||
jinja2 | ||
aiohttp-jinja2 |
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.
This (and probably also jinja2
, and google-cloud-storage
) only need to be in requirements/main.in
because they're runtime dependencies. The things we put in requirements/dev.in
are additional dependencies that get installed in the development/testing environments for running tests. So basically, replicating the runtime dependences here doesn't do anything.
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.
That makes sense now 👍
@@ -3,14 +3,15 @@ | |||
# as implicit dev dependencies with constrained versions. | |||
# | |||
# After editing, update requirements/dev.txt by running: | |||
# make update-deps | |||
# make update |
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.
Just fyi that update
calls make update-deps
, but also re-installs your environment.
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.
Useful to know, thanks!
785bd1b
to
82154ff
Compare
No description provided.