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

Update for eslint 2 #69

Open
mshick opened this issue Feb 23, 2016 · 5 comments
Open

Update for eslint 2 #69

mshick opened this issue Feb 23, 2016 · 5 comments

Comments

@mshick
Copy link

mshick commented Feb 23, 2016

I'm sure this is on your radar, but eslint 2 has a large number of rule name changes, and these configs don't seem to be up-to-date.

@Panda-ref
Copy link

+1

@lkrnac
Copy link
Contributor

lkrnac commented Feb 24, 2016

+1
There's an open pull request for it: #68

@baer
Copy link
Contributor

baer commented Feb 25, 2016

Hey Folks, I'm super sorry something went wrong and I didn't get an email about this issue. This gets a +1 from me too :). Here are the TODO: Items left from the migration guide:

  • Convert no-arrow-condition to use no-confusing-arrow and no-constant-condition
  • Figure out how to set up the new Global Variables to support environments like Node that are not fully compatable with ES-next.
  • Convert all ecmaFeatures to be parser options as noted in the migration guide
  • Move jsx option under the parser options
  • Update eslint:recommended config
  • Update strict rule to use new default

I'll be working on this over the next week or two :)

@lkrnac
Copy link
Contributor

lkrnac commented Feb 27, 2016

I created another PR #71 that should replace #68.
@baer, you are right. I fixed only issues that were breaking my build.

@Snugug
Copy link

Snugug commented May 15, 2016

Any movement on publishing this out of alpha?

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

No branches or pull requests

5 participants