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

Improve package versioning to resolve #7 #10

Closed
wants to merge 2 commits into from

Conversation

1Mark
Copy link
Collaborator

@1Mark 1Mark commented Mar 10, 2022

@1Mark 1Mark marked this pull request as draft March 10, 2022 20:57
@1Mark 1Mark marked this pull request as draft March 10, 2022 20:57
@1Mark
Copy link
Collaborator Author

1Mark commented Mar 10, 2022

I guess we shouldnt allow bump2version to tag for us since we create the tag via https://github.com/loicleyendecker/deeppath/releases/new ?

1 similar comment
@1Mark
Copy link
Collaborator Author

1Mark commented Mar 10, 2022

I guess we shouldnt allow bump2version to tag for us since we create the tag via https://github.com/loicleyendecker/deeppath/releases/new ?

@1Mark 1Mark force-pushed the fix-versioning branch 2 times, most recently from 1ca3b7a to 639285b Compare March 12, 2022 20:18
@1Mark
Copy link
Collaborator Author

1Mark commented Mar 12, 2022

Have a look at https://github.com/1Mark/deeppath/runs/5523775597?check_suite_focus=true
Look at the fourth step bump package version
image

- Add automatic version bumping when a new release is created via
https://github.com/loicleyendecker/deeppath/releases/new
- tag name must be semver and follow va.b.c
- move bump2version config from setup.cfg to .bumpversion.cfg
since it will delete any comments in the cfg files
see c4urself/bump2version#37
[bumpversion]
current_version = 0.1.4
commit = True

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Intentionally dont tag since we will do this via https://github.com/loicleyendecker/deeppath/releases/new

bump2version --new-version ${GITHUB_REF_NAME#v} minor
# must supply either patch minor or major see https://github.com/c4urself/bump2version/issues/244
# but it's an ignored argument argument
git show
Copy link
Collaborator Author

@1Mark 1Mark Mar 12, 2022

Choose a reason for hiding this comment

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

We also need a step to perform a git push at the very end - after pypi upload is successful - but being in a detached head mode in the CI is proving an issue. Going to park this for now :(

@1Mark 1Mark closed this Apr 9, 2022
@1Mark 1Mark deleted the fix-versioning branch April 9, 2022 16:04
@1Mark 1Mark restored the fix-versioning branch April 9, 2022 16:04
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.

2 participants