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: Add ESLint API (refs eslint/rfcs#40) #12939
Conversation
ae3b417
to
ff39b5f
Compare
d394aa6
to
d7276fd
Compare
Closing this for now while I’m out sick, in case someone else can get to it first. |
Reopening in case someone wants to work off of this branch (and so it doesn't get removed from the project board). |
Co-Authored-By: Nicholas C. Zakas <nicholas@nczconsulting.com>
Co-Authored-By: Kai Cataldo <kai@kaicataldo.com>
Thank you for your review! I applied the suggestions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't look like I can officially approve this (maybe because I'm also an author), but LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bravo to both @kaicataldo and @mysticatea 👏 This was an enormous lift, and I'm excited to be able to use the new API. I had some suggestions, mostly around the documentation.
Co-Authored-By: Brandon Mills <btmills@users.noreply.github.com>
Co-Authored-By: Brandon Mills <btmills@users.noreply.github.com>
Co-Authored-By: Brandon Mills <btmills@users.noreply.github.com>
Co-Authored-By: Brandon Mills <btmills@users.noreply.github.com>
Co-Authored-By: Brandon Mills <btmills@users.noreply.github.com>
Co-Authored-By: Brandon Mills <btmills@users.noreply.github.com>
Co-Authored-By: Brandon Mills <btmills@users.noreply.github.com>
Co-Authored-By: Brandon Mills <btmills@users.noreply.github.com>
Co-Authored-By: Brandon Mills <btmills@users.noreply.github.com>
Co-Authored-By: Brandon Mills <btmills@users.noreply.github.com>
Thank you for your review! I applied suggestions to this PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Again, this is great work 👏
@mysticatea Thank you again for picking this up while I was sick ❤️ |
Prerequisites checklist
What is the purpose of this pull request? (put an "X" next to an item)
[ ] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofixing to a rule
[ ] Add a CLI option
[x] Add something to the core
[ ] Other, please explain:
refs eslint/rfcs#40
This is a WIP. I received some initial feedback here and have incorporated that into this PR.
What changes did you make? (Give an overview)
This PR adds the new
ESLint
class, which is an asynchronous wrapper around theCLIEngine
class.Is there anything you'd like reviewers to focus on?
Still a WIP, but as I get closer to completion: are the options/type defs correct?