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

release-start with commitDevelopmentVersionAtStart uses releaseFinishMessage #120

Closed
akuntsch opened this issue Aug 10, 2018 · 8 comments
Closed

Comments

@akuntsch
Copy link

See https://github.com/aleksandr-m/gitflow-maven-plugin/blob/master/src/main/java/com/amashchenko/maven/plugin/gitflow/GitFlowReleaseStartMojo.java#L210

Shouldn't it use the releaseStartMessage?

@raehalme
Copy link
Contributor

releaseStartMessage is used when committing the release version. The line you're referring to commits the next snapshot version and therefore uses the releaseFinishMessage.

@akuntsch
Copy link
Author

I know. But the commit still happens during release-start, so imo it does not make sense to use the releaseFinishMessage when the release is not yet finished.

@raehalme
Copy link
Contributor

The releaseFinishMessage has a value of update for next development version and that's what it does.

@raehalme
Copy link
Contributor

I agree the name of the message is a bit misleading in this context.

@akuntsch
Copy link
Author

Yea, I didn't think about the default releaseStartMessage.

@akuntsch
Copy link
Author

akuntsch commented Aug 13, 2018

So how do we proceed? I think there are multiple options:

  1. change the default commit messages
  2. add another commit message for that special case
  3. leave as is

Maybe we can get @aleksandr-m's opinion on this?

@aleksandr-m
Copy link
Owner

What exactly the problem is? Are you overriding default message? How bad does it looks like with your configuration?

@akuntsch
Copy link
Author

Yes, I am overriding the default messages.

For example if I set the releaseStartMessage to release start and the release finish message to release finish when using commitDevelopmentVersionAtStart, then the commit created on the development branch during mvn gitflow:release-start will have release finish as message, which is pretty misleading.

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

No branches or pull requests

3 participants