-
Notifications
You must be signed in to change notification settings - Fork 395
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
[tests] Refactor tox.ini file #609
Conversation
5d1a820
to
bedcf85
Compare
- We defined specific commands depending on the real command and not on the dependency graph - We removed django testenv special configurations that were no longer used after we added redis{210} in place of redis to the dependency list - We sorted alphabetically the various envs - We do not rely anymore on checksum of tox.ini to invalidate cache
bedcf85
to
75632f8
Compare
ebabac6
to
75632f8
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.
Just one comment, should be good to go after that!
@@ -10,6 +10,15 @@ test_runner: &test_runner | |||
image: datadog/docker-library:dd_trace_py_1_1_0 | |||
env: | |||
TOX_SKIP_DIST: True | |||
restore_cache_step: &restore_cache_step |
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.
We should probably document the usage and motivation behind Environment.CACHE_EXPIRE_HASH
so that it doesn't look like magic or require much digging in the future 😄
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.
Great idea. Will do right now! 👍
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.
@Kyle-Verhoog just pushed an update. Would you please take a look at it? Thanks
the dependency graph
used after we added redis{210} in place of redis to the dependency list