-
-
Notifications
You must be signed in to change notification settings - Fork 88
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
Meta type #185
Meta type #185
Conversation
Any chance y'all can take a look? Thanks... |
Will try to look this weekend or on Monday. Appreciate it
…On Fri, May 15, 2020 at 8:21 PM Brett Zamir ***@***.***> wrote:
Any chance y'all can take a look? Thanks...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#185 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAB4C7YSDDNNZRBKMBZBE3LRRYBERANCNFSM4MF6MCUQ>
.
|
What, no firm promises from the promise plugin? ;-) JK of course... Thanks! |
I'm basically fine with this change, but not really in love with the formatting change (specifically adding trailing commas). If you want to introduce another PR where you bring in prettier that would be fine, but otherwise I'd prefer if you could remove that change and then I'll be fine with merging. |
I guess you mean I could add a separate PR to update Prettier (which starts requiring trailing commas) along with the corresponding linting changes? (Just wanted to be sure you were aware an earlier version of Prettier was already in force.) |
Can use `eslint --fix-type` with meta.type (see https://eslint.org/docs/user-guide/command-line-interface#fix-type )
What is the purpose of this pull request?
What changes did you make? (Give an overview)
meta.type
to all rulesCan use
eslint --fix-type
with meta.type (see https://eslint.org/docs/user-guide/command-line-interface#fix-type )(I'm also working on an eslint badge formatter which allows categorizing rule violations by
meta.type
(i.e., problem, layout, or suggestion), so having this fixed would allow it to be properly categorized.)