-
-
Notifications
You must be signed in to change notification settings - Fork 920
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
infra: configure lint rule array-type #1793
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## next #1793 +/- ##
=======================================
Coverage 99.63% 99.63%
=======================================
Files 2346 2346
Lines 235000 235002 +2
Branches 1133 1133
=======================================
+ Hits 234145 234147 +2
Misses 833 833
Partials 22 22
|
Please add a hint what the new one is^^ I generally agree with adding this rule, I just consider IMO the Is there a reason for the shuffled declaration? |
both are new (?!) |
This is exactly what I also thought! |
This configures the lint rule https://typescript-eslint.io/rules/array-type/
We could decide if we want to configure it with:
{ default: "array-simple" }
or{ default: "array-simple", readonly: "generic" }
The difference would be:
vs