-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[primer] stop overwriting --rcfile
#6897
[primer] stop overwriting --rcfile
#6897
Conversation
Pull Request Test Coverage Report for Build 2470403195
💛 - Coveralls |
The error of the CI is strange, it seems there's a mix up between django and black |
@jacobtylerwalls Did you test this on your local branch to see the time differences? |
058fe87
to
1c51047
Compare
Good point, I hadn't. I just did. I removed Neither of the two projects I'm adding support 3.7 anymore. I'm wondering if we even need that job? |
main
job
f62d06e
to
5b6f229
Compare
main
job--rcfile
The changes in I plan to merge this tonight based on Pierre's approval earlier of the cleanups. I'll move discussion about adding packages to #5359, since I think it's clear we need to look at parallelization instead of just adding packages to a single run. The 3.10 (the faster one!) errored out on CI (on my fork, with adding all packages) with an out of memory error, probably trying to deal with the massive list of messages. The 3.7 run would have no hope. |
Type of Changes
Description
Move everything into the new primer and increase the timeout on themain
job. (3.7 is about 2x as slow as 3.10)Follow up to #6761 and #6753.
Never mind, we will have to discuss that separately.