Skip to content

eslint no single char vars

Owen Ripley requested to merge id-length-min-2 into master

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

Merge request reports