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 new eslint rule format (Fixes #661) #699

Merged
merged 1 commit into from
Jul 27, 2016

Conversation

petersendidit
Copy link
Contributor

The provided automatic transformation from eslint worked great.

Merging this will most likely result in conflicts in all open pull requests.

@ljharb
Copy link
Member

ljharb commented Jul 22, 2016

Awesome! Presumably all the authors of open PRs can run the same transformation and the resulting diffs should be just as useful?

(linking to #686)

@yannickcr yannickcr mentioned this pull request Jul 22, 2016
9 tasks
@petersendidit
Copy link
Contributor Author

Should be, might be a little weird since their branch will be trying to do the same changes as this one. Though its possible that running the tool and then doing a rebase might clean things up for them.

@ljharb
Copy link
Member

ljharb commented Jul 22, 2016

Yes, definitely they'd want to do a rebase. Always be rebasing!

@petersendidit petersendidit force-pushed the NewRuleFormat branch 2 times, most recently from 70645d6 to 7121cbd Compare July 27, 2016 22:10
@yannickcr yannickcr merged commit 10c6f23 into jsx-eslint:master Jul 27, 2016
@yannickcr
Copy link
Member

Merged, thanks!

@petersendidit petersendidit deleted the NewRuleFormat branch August 8, 2016 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants