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

Switch to eslint #42

Merged
merged 1 commit into from Jan 3, 2017
Merged

Switch to eslint #42

merged 1 commit into from Jan 3, 2017

Conversation

edg2s
Copy link
Member

@edg2s edg2s commented Dec 23, 2016

No description provided.

{
"extends": "wikimedia",
"env": {
"browser": true
Copy link
Member

Choose a reason for hiding this comment

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

"node", not "browser".

@Krinkle
Copy link
Member

Krinkle commented Dec 24, 2016

The build is failing since grunt-eslint requires Node 4 or higher as of v19.0.0. I'll make sure Node 4 is supported, and will drop support for Node v0.10. However I'd like to keep support for v0.12 for a little while longer until we go v2.0.

@Krinkle Krinkle added Category: Build About the test, benchmark or release process Type: Enhancement labels Dec 24, 2016
@edg2s
Copy link
Member Author

edg2s commented Dec 24, 2016

should I change to 18.x?

@Krinkle
Copy link
Member

Krinkle commented Dec 24, 2016

@edg2s Might work, yeah.

@edg2s
Copy link
Member Author

edg2s commented Dec 29, 2016

Seems to work

@Krinkle Krinkle merged commit 53584da into wikimedia:master Jan 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Build About the test, benchmark or release process Type: Enhancement
Development

Successfully merging this pull request may close these issues.

None yet

2 participants