-
Notifications
You must be signed in to change notification settings - Fork 160
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
Remove immutable state vars from var-name-mixedcase #270
Comments
I ran into this as well. I would actually suggest that I don't think new rules or options are necessary. |
I guess some people still like to make So I think having the extra |
yes please! |
How are we on this? |
see this PR |
Like
constant
state vars I also like tosnake-case
myimmutable
state vars because it makes it easier to spot in my code where I am relying on immutable values.However, currently this clashes with two rules I also use:
var-name-mixedcase
forces me to also makeimmutable
mixed-caseconst-name-snakecase
does not force me to makeimmutable
snake-caseI would like to propose one of the following alterantives as feature requests:
Alternative 1: New rule:
immutable-name-snakecase
that when turned on overrides thevar-name-mixedcase
rules.Alternative 2:
strict
option forvar-name-mixedcase
which whenfalse
does not includeimmutable
vars &&strict
option forconst-name-snakecase
that whentrue
also forcesimmutable
vars to besnakecase
The text was updated successfully, but these errors were encountered: