Skip to content

Feature branch with jira key #337

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

Merged

Conversation

JakeGinnivan
Copy link
Contributor

Replacing #327 with a possible solution to the failing test

orjan and others added 2 commits December 18, 2014 22:59
This is just a test to reproduce my issue with feature branches based on
the jira key e.g. feature/JIRA-123-a-description.
@JakeGinnivan
Copy link
Contributor Author

While looking into this issue I noticed that we could have SemVer which has a - in the tag. These should be replaced with .. We probably need to look into this more and strip out all .'s for SemVer v1 version numbers.

@JakeGinnivan
Copy link
Contributor Author

It also brings up an interesting issue about the difference between GitFlow and GitHubFlow.

For branch feature/JIRA-123
GitFlow: 1.1.0-JIRA.123+5
GitHubFlow: 1.0.1-feature.JIRA.123+5

Thoughts @orjan @andreasohlund @SimonCropp @gep13

@orjan
Copy link
Contributor

orjan commented Jan 6, 2015

I'm not sure that the jira identifier "JIRA-123" should be splitted since an identifier might contain - http://semver.org/ [0-9A-Za-z-] so I would prefer
1.1.0-JIRA-123.5

@JakeGinnivan
Copy link
Contributor Author

@orjan you are right. I didn't realise that hyphens were allowed in a pre-release tag. That seems strange to me, but that means I do not have to replace the - with .

@JakeGinnivan
Copy link
Contributor Author

@orjan as for the .5 vs +5

That will be a configuration option in the new version hopefully. You will have to use the variables or format the version manually at the moment.

@orjan
Copy link
Contributor

orjan commented Jan 6, 2015

@JakeGinnivan great news regarding +5 .5 I'll guess we're all having different use cases.

@orjan
Copy link
Contributor

orjan commented Jan 6, 2015

@JakeGinnivan both GitFlow and GitHubFlow is looking good but I cannot see why the should differ.
I'll guess #325 will handle customised prerelease tags?

@gep13
Copy link
Member

gep13 commented Jan 6, 2015

@JakeGinnivan so is the current branch tagged at 1.0.0, or the NextVersion.txt contains 1.0.0, or in the YAML file it has the 1.0.0?

In which case, GitFlow is returning correctly, right?

Just want to make sure we are on the same page 😸

@JakeGinnivan
Copy link
Contributor Author

GitHubFlow bumps patch, GitFlow bumps minor as patch is done with hotfix branches.

@gep13
Copy link
Member

gep13 commented Jan 6, 2015

GitHubFlow bumps patch, GitFlow bumps minor as patch is done with hotfix branches.

Now that, I should have known, but then I don't really use GitHubFlow 😸 So what is the difference that you are referring to? The inclusion of the word feature?

@JakeGinnivan
Copy link
Contributor Author

Oh. yeah

GitFlow understands feature branches. GithubFlow doesn't.

@gep13
Copy link
Member

gep13 commented Jan 6, 2015

Oh. yeah

GitFlow understands feature branches. GithubFlow doesn't.

Right, sorry, now I follow! I thought you were saying that there was a problem with the differences being there. But you are saying that these differences are expected, and you were just highlighting them.

I will step away from the keyboard now...

@JakeGinnivan
Copy link
Contributor Author

Updated based on feedback.

For branch feature/JIRA-123
GitFlow: 1.1.0-JIRA-123+5
GitHubFlow: 1.0.1-JIRA-123+5 - got rid of the feature. seemed silly

@andreasohlund
Copy link
Contributor

+1

On Wed, Jan 7, 2015 at 10:22 AM, Jake Ginnivan [email protected]
wrote:

Updated based on feedback.

For branch feature/JIRA-123
GitFlow: 1.1.0-JIRA-123+5
GitHubFlow: 1.0.1-JIRA-123+5 - got rid of the feature. seemed silly


Reply to this email directly or view it on GitHub
#337 (comment)
.

JakeGinnivan added a commit that referenced this pull request Jan 7, 2015
@JakeGinnivan JakeGinnivan merged commit 308aca8 into GitTools:master Jan 7, 2015
@JakeGinnivan JakeGinnivan deleted the feature-branch-with-jira-key branch January 7, 2015 17:16
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.

4 participants