Skip to content
This repository has been archived by the owner on Apr 14, 2022. It is now read-only.

Merge CI improvements upstream #114

Open
pquentin opened this issue Aug 20, 2019 · 2 comments
Open

Merge CI improvements upstream #114

pquentin opened this issue Aug 20, 2019 · 2 comments

Comments

@pquentin
Copy link
Member

pquentin commented Aug 20, 2019

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:

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 (using rg '" "')
  • Use cls in setup_class
  • Remove 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
@RatanShreshtha
Copy link
Member

I think we should keep pytest-random-order and configure it in our CI, I will be happy to open a PR for that.

@pquentin
Copy link
Member Author

Thanks, that would be useful indeed 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants