-
-
Notifications
You must be signed in to change notification settings - Fork 339
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
feat(linter): @typescript-eslint/consistent-indexed-object-style #3126
feat(linter): @typescript-eslint/consistent-indexed-object-style #3126
Conversation
CodSpeed Performance ReportMerging #3126 will not alter performanceComparing Summary
|
crates/oxc_linter/src/rules/typescript/consistent_indexed_object_style.rs
Outdated
Show resolved
Hide resolved
crates/oxc_linter/src/rules/typescript/consistent_indexed_object_style.rs
Outdated
Show resolved
Hide resolved
crates/oxc_linter/src/rules/typescript/consistent_indexed_object_style.rs
Outdated
Show resolved
Hide resolved
crates/oxc_linter/src/rules/typescript/consistent_indexed_object_style.rs
Outdated
Show resolved
Hide resolved
crates/oxc_linter/src/rules/typescript/consistent_indexed_object_style.rs
Outdated
Show resolved
Hide resolved
1834331
to
8efa6aa
Compare
8efa6aa
to
5a07354
Compare
Thanks to your effort on this rule, take your time! |
Hey, @mysteryven. After some more tinkering I think finally did it. Have another look when you get a chance and let me know what you think. P.S. I managed to implement a fixer only for one of the settings. I will try to implement it for the other one as well but if not, we can log a separate task. |
I gonna to merge this first, let's iterate it later :) And thanks again for all your work. |
I forgot to update the docs in #3126.
implements https://typescript-eslint.io/rules/consistent-indexed-object-style/