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

should get exact version from tag #211

Merged
merged 2 commits into from
Jul 16, 2014
Merged

should get exact version from tag #211

merged 2 commits into from
Jul 16, 2014

Conversation

SimonCropp
Copy link
Contributor

No description provided.

@SimonCropp
Copy link
Contributor Author

@andreasohlund @JakeGinnivan @distantcam does this look like a valid unit test?

@JakeGinnivan
Copy link
Contributor

Looks right to me

"Number": 2
},
"BuildMetaData": {
"CommitsSinceTag": 1,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be 0?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It probably should be.. I think the wiki is wrong in this regard sigh

Thoughts

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@distantcam @JakeGinnivan issue raised here #213

@SimonCropp
Copy link
Contributor Author

note this is only for GV. fixed by @distantcam for GHF in this commit dd63885

@SimonCropp
Copy link
Contributor Author

@JakeGinnivan @distantcam @andreasohlund code review is anyone has time

@SimonCropp SimonCropp modified the milestone: 1.1 Jul 15, 2014
JakeGinnivan added a commit that referenced this pull request Jul 16, 2014
should get exact version from tag
@JakeGinnivan JakeGinnivan merged commit 58d7af2 into master Jul 16, 2014
@JakeGinnivan JakeGinnivan deleted the exactTagOnRelease branch July 16, 2014 06:36
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