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.

Sign up for GitHub

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

Jump to bottom

Switch to eslint #155

Merged
merged 1 commit into from
Oct 7, 2016
Merged

Switch to eslint #155

merged 1 commit into from
Oct 7, 2016

Conversation

cowboyd
Copy link
Collaborator

@cowboyd cowboyd commented Sep 30, 2016

eslint is the future of JS linting. This also adds support for the prefer-let.

@cowboyd
Copy link
Collaborator Author

cowboyd commented Sep 30, 2016

I had to add the select helper to the globals list, which might be better done inside the tests .eslintrc.js

@Robdel12 Robdel12 modified the milestone: v3.0.0 Oct 7, 2016
@cowboyd @Robdel12
Switch to eslint
e4a983a 
eslint is the future of JS linting. This also adds support for the
`prefer-let`.
@Robdel12 Robdel12 force-pushed the switch-to-eslint branch 2 times, most recently from 4dc1cd3 to e4a983a Compare October 7, 2016 02:33
@Robdel12 Robdel12 merged commit c493c76 into master Oct 7, 2016
@Robdel12 Robdel12 deleted the switch-to-eslint branch December 27, 2016 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v3.0.0
Development

Successfully merging this pull request may close these issues.
2 participants
@cowboyd @Robdel12