Skip to content

Alias bump2version as bumpversion.#2

Merged
ekohl merged 1 commit into
c4urself:masterfrom
the-allanc:bumpversion-alias
Sep 25, 2017
Merged

Alias bump2version as bumpversion.#2
ekohl merged 1 commit into
c4urself:masterfrom
the-allanc:bumpversion-alias

Conversation

@the-allanc
Copy link
Copy Markdown
Contributor

I can't imagine anyone who will install bump2version but will still want to use the original version of bumpversion (other than to compare the two). So even though we can allow people to install whichever version they want, how about we get bump2version to also alias itself to bumpversion?

When I installed it, I expected it to override bumpversion, not to sit alongside it - so out of habit, I invoked bumpversion patch and didn't get an annotated tag like I was expecting.

@ekohl
Copy link
Copy Markdown
Collaborator

ekohl commented Sep 20, 2017

I wonder how Python deals with 2 packages trying to create the same script but in general this makes sense to me. 👍

@c4urself
Copy link
Copy Markdown
Owner

I'm OK with this as well -- @the-allanc mind creating a PR?

@ekohl
Copy link
Copy Markdown
Collaborator

ekohl commented Sep 21, 2017

@c4urself this is a PR?

@c4urself
Copy link
Copy Markdown
Owner

ugh -- early morning -- sorry about this :(

@ekohl ekohl merged commit be202ea into c4urself:master Sep 25, 2017
@ekohl
Copy link
Copy Markdown
Collaborator

ekohl commented Sep 25, 2017

@the-allanc thanks!

@the-allanc
Copy link
Copy Markdown
Contributor Author

Could we get this into a release?

@the-allanc the-allanc deleted the bumpversion-alias branch September 27, 2017 15:21
@ekohl
Copy link
Copy Markdown
Collaborator

ekohl commented Sep 27, 2017

I have no permission to push a release to pypi. @c4urself?

@c4urself
Copy link
Copy Markdown
Owner

@the-allanc @ekohl done

@the-allanc
Copy link
Copy Markdown
Contributor Author

Thanks guys!

florisla pushed a commit that referenced this pull request Jan 19, 2021
Change format on disk + cosmetics in the MarkDown
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.

4 participants