Skip to content

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

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.

5 participants