eslint no single char vars
What is the new behaviour?
added id-length
rule to eslint. Variable names must now be minimum two characters long.
Additional Context
- Related to #
- Anything else that is useful for the reviewer to know
Suggested methods of validation
- How can this MR be validated once it is deployed?
MR Author Checklist
Please check off all that apply:
-
Added documentation/comments for any API endpoints, reusable code, and complex code -
Information is being logged for debugging -
All new variables are set -
Any new manual resources are provisioned -
If feature work, is targetted to feature branch -
Code follows code standards -
Added unit tests - required for: - New projects
- New features in legacy projects
- Major refactors in legacy projects
- Bug fixes, small refactors and easily testable code (not required, at developer's discretion)
Notes: (Anything else the reviewer needs to know about the points above)
Steps To Take After Creating the MR
-
Move ticket to "Dev Review"
Reviewer Checklist
-
Check that the right branch is targeted (master/feature) -
Confirm that the MR author followed all needed steps above