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

deps: peg grunt to ~0.4 #3092

Merged
merged 1 commit into from Apr 6, 2016
Merged

deps: peg grunt to ~0.4 #3092

merged 1 commit into from Apr 6, 2016

Conversation

MylesBorins
Copy link
Contributor

Dev deps cannot pass with grunt > 0.4

1.0 will fail without deps themselves updating. PEgging the grunt dep
ensures that npm install will work without error

fixes #3091

Dev deps cannot pass with grunt > 0.4

1.0 will fail without deps themselves updating. PEgging the grunt dep
ensures that npm install will work without error

fixes moment#3091
@MylesBorins
Copy link
Contributor Author

ping @ichernev

@ichernev
Copy link
Contributor

ichernev commented Apr 6, 2016

@thealphanerd thanks, looks good.

@ichernev ichernev merged commit c2c2472 into moment:develop Apr 6, 2016
@ichernev
Copy link
Contributor

ichernev commented Apr 6, 2016

@thealphanerd do you also need a new release tag, or just develop branch is fine.

@MylesBorins
Copy link
Contributor Author

a release would be preferable. This breaks us being able to test in ci

@MylesBorins
Copy link
Contributor Author

@ichernev quick ping about a release tag. Just had a team reach out about this module failing their smoke tests

@ichernev
Copy link
Contributor

I'll try to release this weekend.

@ichernev
Copy link
Contributor

@thealphanerd 2.13.0 is released in npm, hope its all good on your end now.

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.

npm install broken by new grunt
2 participants