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

Fix incorrect git ref version choices; See #12 #13

Merged
merged 1 commit into from
Jan 21, 2015
Merged

Fix incorrect git ref version choices; See #12 #13

merged 1 commit into from
Jan 21, 2015

Conversation

tarfu
Copy link
Contributor

@tarfu tarfu commented Jan 21, 2015

Sorting with always the same Version does not really make sense.
We parsed the version for the ref but never used it for sorting or anything else but for choosing if it is the right major version

Sorting with always the same Version does not really make sense.
We parsed the version for the ref but never used it for sorting or anything else but for choosing if it is the right major version
@slimsag slimsag modified the milestone: 2.0.1 Jan 21, 2015
@slimsag
Copy link
Owner

slimsag commented Jan 21, 2015

I really appreciate you tracking this down. I will try my best to get 2.0.1 out by today.

I am going to see if I can make some tests before merging this to identify when it fails so this doesn't occur in the future.

@slimsag slimsag changed the title Issue #12: Fixing false Version chosen Fix incorrect git ref version choices; See #12 Jan 21, 2015
slimsag added a commit that referenced this pull request Jan 21, 2015
Fix incorrect git ref version choices; See #12
@slimsag slimsag merged commit 993ebea into slimsag:master Jan 21, 2015
@slimsag
Copy link
Owner

slimsag commented Jan 21, 2015

Please also see http://azul3d.org/doc/contributing.html#attribution, if you would like to be properly attributed for you work here (which I really appreciate, by the way!).

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

2 participants