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

Add grunt release:[releasetype] task #88

Merged
merged 1 commit into from
Aug 4, 2015
Merged

Add grunt release:[releasetype] task #88

merged 1 commit into from
Aug 4, 2015

Conversation

SimonWoolf
Copy link
Member

Usage: grunt release:patch to increment the patch level, compile everything, commit the generated files, and tag that commit with the release version.

Can also use minor, major, prepatch, prerelease etc. instead of patch.

@paddybyers
Copy link
Member

LGTM but how has this been tested?

@SimonWoolf
Copy link
Member Author

By me trying it locally - it doesn't push anything or use ably-env at all, just creates the commit, so easy enough to try out and revert (don't forget to delete the new tag it creates, removing the commit it points to doesn't do that)

@paddybyers
Copy link
Member

/cc @mattheworiordan any comments?

"karma-chrome-launcher": "~0.1",
"karma-firefox-launcher": "~0.1",
Copy link
Member

Choose a reason for hiding this comment

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

Why is this removed?

Copy link
Member Author

Choose a reason for hiding this comment

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

It wasn't. It was moved down to line 32, karma-nodeunit was moved from 35 to 33, karma-cli from 27 to 30. npm sorts the array into alphabetical order automatically. Presumably it had got out of alphabetical order due to entries being manually added

Copy link
Member

Choose a reason for hiding this comment

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

Ah yes, sorry, reviewed in haste

@mattheworiordan
Copy link
Member

👍

@SimonWoolf SimonWoolf merged commit 1e0a81c into master Aug 4, 2015
SimonWoolf added a commit that referenced this pull request Aug 4, 2015
@paddybyers paddybyers deleted the grunt-release branch August 4, 2015 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants