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

Move to setuptools_scm instead of bumpversion. #403

Merged
merged 1 commit into from May 14, 2021
Merged

Conversation

jezdez
Copy link
Member

@jezdez jezdez commented May 13, 2021

This is mostly things I forgot to add in #397.

@jezdez
Copy link
Member Author

jezdez commented May 13, 2021

@davesque @Andrew-Chen-Wang This is to make it simpler to ship updates by simply creating and push Git tags (or GitHub releases that do that) to trigger the release process to the Jazzband staging package index.

bumpversion - while useful to update static version information - is not super compatible with PyPA's setuptools_scm and the version scheme that was in place wasn't either. As such I've simply removed it for now.

@jezdez jezdez requested review from Andrew-Chen-Wang and davesque and removed request for Andrew-Chen-Wang May 13, 2021 17:17
@jezdez jezdez added this to In progress in Migrate to GitHub Actions via automation May 13, 2021
@jezdez jezdez moved this from In progress to In review in Migrate to GitHub Actions May 13, 2021
@davesque
Copy link
Member

Hey @jezdez ! Thanks so much for doing this! I think this is all fine but I'll take a more thorough look when I get home!

Copy link
Member

@davesque davesque left a comment

Choose a reason for hiding this comment

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

🚢

Migrate to GitHub Actions automation moved this from In review to Reviewed May 14, 2021
@jezdez jezdez merged commit aaebe1a into master May 14, 2021
Migrate to GitHub Actions automation moved this from Reviewed to Done May 14, 2021
@jezdez jezdez deleted the setuptools-scm branch May 14, 2021 07:27
@Andrew-Chen-Wang
Copy link
Member

Thanks @jezdez for upgrading and working on this repo!

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

Successfully merging this pull request may close these issues.

None yet

3 participants