bump2version icon indicating copy to clipboard operation
bump2version copied to clipboard

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

Open twcurrie opened this issue 4 years ago • 4 comments

Adds --show option referenced in these issues:

  • https://github.com/c4urself/bump2version/issues/57
  • https://github.com/c4urself/bump2version/issues/181

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.

twcurrie avatar Apr 05 '21 16:04 twcurrie

This is long overdue, thanks for contributing this.

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

florisla avatar Jul 20 '21 10:07 florisla

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.

clbarnes avatar Aug 20 '21 11:08 clbarnes

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

twcurrie avatar Aug 24 '21 03:08 twcurrie

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

florisla avatar Mar 02 '22 14:03 florisla