Merge CI improvements upstream
pquentin opened this issue · comments
I think I prefer to wait until we've caught up with upstream to do this, but I still want to record the list, that I will continue to update:
Fix TLS 1.3 tests when usingssl
from the standard library (a1fa2c2)Bump macOS Python versions (important to test 3.7.4 with the above fix)Skip one PyPy 2 test (#110)Remove newlines afterwith
statements (done as part of #119)Python 2 coverage uploads- setup.cfg
Things that we don't get right either:
- Is the no brotli test running somewhere?
- Can we drop the warnings configuration when launching pytest with the latest pytest warning changes? This would allow to use --disable-warnings
Fix lines merged by black (usingrg '" "'
)Use cls in setup_classRemove useless proxy_from_url redirection(it's part of the public API)
Things that are specific to us:
- We no longer use pytest-random-order: either configure it or drop it
- Remove enforce_content_length=False
- AppVeyor does not upload its coverage results to codecov
I think we should keep pytest-random-order and configure it in our CI, I will be happy to open a PR for that.
Thanks, that would be useful indeed 👍