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

Combine CI workflows for testing and release upload to PyPI #291

Merged

Conversation

GenevieveBuckley
Copy link
Collaborator

@GenevieveBuckley GenevieveBuckley commented Mar 14, 2023

Closes #147

We want the tests to run and pass before a new release is uploaded to PyPI. This will be very useful, especially since activity on this repo is infrequent, so it's a good idea to re-run the whole test suite before releasing.

Combining the two CI workflows into the same file makes this easy, since now the PyPI upload is an optional extra step if the tests all pass and a tagged version has been created (either with git --tag v2023.MM.DD or by making a release from the github releases page).

We'll be able to see whether any of the tests fail and why in the github actions tab for this repository. It makes it much easier to create a release candidate tag, check the tests pass and it's uploaded to PyPI, then make the official release.

…tests to run and pass before a release is uploaded)
@GenevieveBuckley
Copy link
Collaborator Author

The new combined CI checks are passing, and we'll see how smoothly it performs at the next release (we should do that soon).

@GenevieveBuckley GenevieveBuckley merged commit d25a129 into dask:main Mar 14, 2023
@jakirkham
Copy link
Member

Awesome! Thanks Genevieve 🙏

Agree it would be good to do a release soon

@GenevieveBuckley GenevieveBuckley deleted the test-before-pypi-releases branch March 27, 2023 01:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Running test suite as part of PyPI release
2 participants