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: Build wheels first in development version tests #31

Merged
merged 7 commits into from
Sep 13, 2024

Conversation

garrison
Copy link
Member

This is an experiment to build wheels first, then run tox in parallel.

@garrison garrison added the ci-cd Related to CI/CD pipeline label Sep 12, 2024
@coveralls
Copy link

coveralls commented Sep 12, 2024

Pull Request Test Coverage Report for Build 10840695662

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 10832955808: 0.0%
Covered Lines: 297
Relevant Lines: 297

💛 - Coveralls

@garrison garrison marked this pull request as ready for review September 13, 2024 00:39
@garrison
Copy link
Member Author

garrison commented Sep 13, 2024

I like this a lot, the way it turned out. It's a lot more transparent than just letting tox handle everything, and it enables parallelization.

@garrison garrison changed the title Build wheels first in development version tests ci: Build wheels first in development version tests Sep 13, 2024
Copy link
Member

@mrossinek mrossinek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I originally refrained from doing this as I did not want to make any assumptions around the build process. But I suppose the build tools have come a long way where this seems straight forward and transferable enough 👍

@mrossinek mrossinek merged commit b8089f3 into main Sep 13, 2024
15 checks passed
@mrossinek mrossinek deleted the dev-tests-wheel-first branch September 13, 2024 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-cd Related to CI/CD pipeline
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants