Add forceUpperCaseConst option to prefer-let#84
Conversation
|
@cowboyd if you don’t need it, I can create a fork |
Package Changes Through 7d03891No changes. Add a change file through the GitHub UI by following this link. Read about change files or the docs at github.com/jbolda/covector |
|
This looks good to me. Can you add a something to the README to explain this option? |
|
Docs were updated 7d03891 |
|
Thanks for quick reaction! |
|
No worries. Although totally forgot to add a changeset, so it did not generate a release PR. Any chance you could create one? Here is an example from the last PR https://github.com/thefrontside/javascript/pull/78/changes#diff-f99018ad680c1619bbdcf2df7d4ef0b3bf07a4407fa0127d6097c1918a637e31
|
|
Done #85 |
Motivation
I am using
letfor all variables andconstfor real logical constant (like config):Approach
I added
forceUpperCaseConstoption toprefer-letwhich will allow to useconstonly for top-level vars with specific name pattern.Alternate Designs
We can enable this option by design if you are also using the same logic.
Possible Drawbacks or Risks
I don’t see any
TODOs and Open Questions
I avoid changing
packages/eslint-plugin-prefer-let/docs/rules/prefer-let.mdbecause it was already empty.Learning
N/A
Screenshots
N/A