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

[New] add eslint 5 #296

Merged
merged 4 commits into from Jun 24, 2018
Merged

[New] add eslint 5 #296

merged 4 commits into from Jun 24, 2018

Conversation

ljharb
Copy link
Collaborator

@ljharb ljharb commented Jun 24, 2018 •
edited

This adds eslint v5 support, and cleans up the travis yml a bit.

(linking back to airbnb/javascript#1834)

@ljharb ljharb mentioned this pull request Jun 24, 2018
Closed
21 tasks
ta2edchimp
ta2edchimp approved these changes Jun 24, 2018
View reviewed changes
Copy link
Collaborator

@ta2edchimp ta2edchimp left a comment

Choose a reason for hiding this comment

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

Thanks for taking care of this!

@ljharb ljharb self-assigned this Jun 24, 2018
@ljharb ljharb force-pushed the eslint_5 branch 4 times, most recently from 3a69379 to 8066cc2 Compare June 24, 2018 17:24
@ljharb ljharb merged commit 4aa5160 into sarbbottam:master Jun 24, 2018
@ljharb ljharb deleted the eslint_5 branch June 24, 2018 17:26
@ljharb ljharb mentioned this pull request Jun 25, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ta2edchimp ta2edchimp ta2edchimp approved these changes

@sarbbottam sarbbottam Awaiting requested review from sarbbottam

Assignees

@ljharb ljharb

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@ljharb @ta2edchimp