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

Fixes to scripts #168

Merged
merged 2 commits into from Sep 30, 2015
Merged

Fixes to scripts #168

merged 2 commits into from Sep 30, 2015

Conversation

quis
Copy link
Contributor

@quis quis commented Sep 29, 2015

Use git describe to find previous version

git tag returns tags alphabetically, not by date.

git describe is the appropriate command, thanks Stack Overflow: http://stackoverflow.com/a/14337129/147318

Remove redundant line

-1

This variable was getting set but not used anywhere
`git tag` returns tags alphabetically, not by date.

`git describe` is the appropriate command, thanks Stack Overflow:
http://stackoverflow.com/a/14337129/147318
@tombye
Copy link
Contributor

tombye commented Sep 30, 2015

👍

quis added a commit that referenced this pull request Sep 30, 2015
@quis quis merged commit 4abff83 into master Sep 30, 2015
@quis quis deleted the scripts-fixes branch October 21, 2015 14:41
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