c4urself / bump2version

Version-bump your software with a single command
https://pypi.python.org/pypi/bump2version
MIT License
1.05k stars 134 forks source link

Adding --show option for CI/CD processes to consume #209

Open twcurrie opened 3 years ago

twcurrie commented 3 years ago

Adds --show option referenced in these issues:

I also strongly desire this feature, so I aimed for simplicity and minimal change on this add-on, and tried to match the coding style as best as possible. I defer to the maintainer should they have stronger opinions on implementation 😉 , but I believe there is strong interest in this feature overall.

florisla commented 3 years ago

This is long overdue, thanks for contributing this.

If I find the time, I will review this and include it in the next release.

clbarnes commented 3 years ago

Might it be better to use print rather than logging to output the version? That would play nicer with scripting, is simpler, and more expected, as logging writes to stderr and should be used to communicate with the user, where print uses stdout and represents a program's primary output.

twcurrie commented 3 years ago

@clbarnes I don't have a strong opinion on that, but it's a trivial change if desired - @florisla thoughts?

florisla commented 2 years ago

I agree that the --show output should go to stdout.