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

make "git flow release" tag master #325

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

JasonWoof
Copy link

git flow release was tagging the last commit in the release branch instead
of master.

This commit makes it tag master (after the merge) instead.

I believe this commit restores the original and intended behavior, which
appears to be changed (probably unintentionally) in commit afb191f.

``git flow release`` was tagging the last commit in the release branch instead
of master.

This commit makes it tag master (after the merge) instead.

I believe this commit restores the original and intended behavior, which
appears to be changed (probably unintentionally) in commit afb191f.
@jasmarc
Copy link

jasmarc commented Jul 16, 2013

+1 I'd like to see this implemented as well. Is there a more formal way of +1ing this?

Was this really the original behavior? Because it's what I expected and was disappointed to find out it's not the current behavior.

@SparK-Cruz
Copy link

+1
I want this working too.

@mmcgahan
Copy link

+1

3 similar comments
@ByFe
Copy link

ByFe commented Jul 31, 2013

+1

@cherrydev
Copy link

+1

@adben
Copy link

adben commented Sep 19, 2013

+1

@petervanderdoes
Copy link

This is implemented in my fork git-flow (AVH Edition)
My fork has diverged to much to make this into a pull request for nvie's gitflow,

Checkout the changelog for more information about bugfixes and new features implemented in my fork.

@cherrydev
Copy link

Is there some way we could get @nvie to comment on if this current behaviour is intended or if this is a bug?

@jnardone
Copy link

Agreed. This needs attention; it breaks our current workflow to use versions based on this. :(

@kaimi
Copy link

kaimi commented Oct 15, 2013

👍

2 similar comments
@micky2be
Copy link

micky2be commented Feb 4, 2014

👍

@kamilchm
Copy link

kamilchm commented Nov 6, 2014

👍

@jawshooah
Copy link

Dudes, this repo is pretty much dead in the water (latest commit was in 2012). If you're looking for active maintenance, check out @petervanderdoes's fork. It heals what ails you.

@ghost
Copy link

ghost commented Nov 6, 2014

@jawshooah
hat-tip

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.

None yet