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 "use strict" to all gruntfiles across projects #94

Closed
ajpiano opened this issue Oct 16, 2012 · 3 comments
Closed

Add "use strict" to all gruntfiles across projects #94

ajpiano opened this issue Oct 16, 2012 · 3 comments
Assignees

Comments

@ajpiano
Copy link
Member

ajpiano commented Oct 16, 2012

Grunt 0.3.17 now requires any project that is linting the gruntfile to include a "use strict" pragma as the first line, we should add one to any project that's linting the gruntfile (or all of them, really). @ronchalant offered to take care of this :)

@rwaldron
Copy link
Member

We literally just deleted this from our grunt.js because it's incorrect.

@ronchalant
Copy link

@rwldrn I noticed that some of the grunt.js have "use strict" incorrectly - e.g. the learn site simply had

use strict;

at the head of the file. I'm fixing them as well, and will skip core leaving it to you

@danheberden
Copy link
Member

added this issue to any remaining sites not pending pull request - @ronchalant if you have more pull requests i'd reference those specific issues per repo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants