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 coffee-script dependency version #25

Merged
merged 1 commit into from
Mar 13, 2017
Merged

Fix coffee-script dependency version #25

merged 1 commit into from
Mar 13, 2017

Conversation

systemist
Copy link
Contributor

coffee-loader uses v3SourceMap, which was introduced in coffee-script@1.6.0, so this makes that dependency explicit. This was broken in projects that already had a dependency on coffee-script >1.0 <= 1.6.0, since that met the peer dependency, yet threw errors on compile.

@michael-ciniawsky
Copy link
Member

@jacobthemyth Please close and reopen the PR to trigger the CLA Bot again. Thx in advance 😛

package.json Outdated
@@ -1,13 +1,13 @@
{
"name": "coffee-loader",
"version": "0.7.2",
"version": "0.7.3",
Copy link
Member

Choose a reason for hiding this comment

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

Please don't 😛 npm version and release will be always done by us.

coffee-loader uses `v3SourceMap`, which was introduced in
`coffee-script@1.6.0`, so this makes that dependency explicit.
@jsf-clabot
Copy link

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

@systemist systemist closed this Mar 13, 2017
@systemist systemist reopened this Mar 13, 2017
Copy link
Member

@michael-ciniawsky michael-ciniawsky left a comment

Choose a reason for hiding this comment

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

@jacobthemyth Thx

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

3 participants