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

Remove use of 'package' as a variable name in install.js #936

Merged
merged 1 commit into from May 8, 2015

Conversation

apiv
Copy link

@apiv apiv commented May 7, 2015

package is a reserved word in strict mode. Therefore, node-sass cannot be packaged by a bundler that forces strict mode.

This pull request removes the package keyword, and renames the variable to pkg, as it is in scripts/build.js.

@xzyfer
Copy link
Contributor

xzyfer commented May 8, 2015

Thanks @ajp427 !

@xzyfer xzyfer added this to the next.patch milestone May 8, 2015
xzyfer added a commit that referenced this pull request May 8, 2015
Remove use of 'package' as a variable name in install.js
@xzyfer xzyfer merged commit a5ff5d6 into sass:master May 8, 2015
@xzyfer xzyfer modified the milestones: next.patch, v3.1.0 May 13, 2015
jiongle1 pushed a commit to scantist-ossops-m2/node-sass that referenced this pull request Apr 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants