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

Need to skip 'master' branch build after gitflow:release... #135

Closed
kitplummer opened this issue Oct 19, 2018 · 9 comments
Closed

Need to skip 'master' branch build after gitflow:release... #135

kitplummer opened this issue Oct 19, 2018 · 9 comments

Comments

@kitplummer
Copy link

I have a feeling we're close, but are just missing something simple.

We run: mvn -B gitflow:release and everything works as expected, we get our 'develop' and 'master' branches updated as expected. However, when the push to 'master' happens it triggers another build. We're trying to use Bitbucket.org's Pipelines - which provide a "skip" ability by using [skip CI] as the commit message. We've tried all kinds of combos of defining the commit messages in the pom configuration. However, there isn't a commit message for the merge from develop to master - when the push happens during the release so we end up unable to actual skip it.

Any insight/help would be so greatly appreciated. We've poked at rebasing and fast-forwards to no avail too.

TIA.

@jjlharrison
Copy link

I'm working on a pull-request to allow custom merge commit messages. Would that solve your problem?

@kitplummer
Copy link
Author

I believe so!

@aleksandr-m
Copy link
Owner

@jjlharrison I think most users are ok with default git merge messages and they can be customized in git config. Custom messages in this plugin would override them. How about create blank merge commit messages in CommitMessages and if they are blank then use default git messages.

@jjlharrison
Copy link

@jjlharrison I think most users are ok with default git merge messages and they can be customized in git config. Custom messages in this plugin would override them. How about create blank merge commit messages in CommitMessages and if they are blank then use default git messages.

Sure, will do. 👍🏻

@kitplummer
Copy link
Author

That'd be awesome. Thanks guys, looking forward to getting this working.

@aleksandr-m
Copy link
Owner

@jjlharrison Any progress?

@aleksandr-m
Copy link
Owner

@kitplummer I've pushed some changes. Can you take a look.
Do you need some parameters in merge commit message e.g. branch names?

@kitplummer
Copy link
Author

Cool! Looks like that'll work. I don't believe we would.

@aleksandr-m
Copy link
Owner

@kitplummer Great! 1.11.0 is released.

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