Skip to content
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

ci: speed up pytest collect #370

Merged
merged 6 commits into from
Dec 5, 2021
Merged

Conversation

redeboer
Copy link
Member

@redeboer redeboer commented Dec 5, 2021

Import expensive modules in the tests inline so that pytest --collect-only is faster.

@redeboer redeboer added the 🖱️ DX Improvements to the Developer Experience label Dec 5, 2021
@redeboer redeboer added this to the 0.4.0 milestone Dec 5, 2021
@redeboer redeboer self-assigned this Dec 5, 2021
@redeboer redeboer enabled auto-merge (squash) December 5, 2021 13:44
@redeboer redeboer enabled auto-merge (squash) December 5, 2021 13:45
@redeboer redeboer merged commit e1f0034 into ComPWA:main Dec 5, 2021
@redeboer redeboer deleted the speed-up-pytest branch December 5, 2021 19:45
redeboer added a commit that referenced this pull request Dec 7, 2021
* ci: merge tox cov job into py job
* ci: simplify tox pydeps job
* fix: include doctests in coverage computation
* fix: remove histogram from tox benchmark job
  (causes problems with pygal and pygaljs)
* test: speed up pytest collect with inline imports
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🖱️ DX Improvements to the Developer Experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant