Uploaded image for project: 'Node.js Driver'
  1. Node.js Driver
  2. NODE-4319

Add eslint rule preventing variable shadowing to eslint configurations

XMLWordPrintableJSON

    • Type: Icon: Improvement Improvement
    • Resolution: Unresolved
    • Priority: Icon: Minor - P4 Minor - P4
    • None
    • Affects Version/s: None
    • Component/s: Tooling

      As a node driver engineer,
      I want to prevent variable shadowing except where absolutely necessary,
      So that future engineers understand that shadowing is intentional in the places it exists in our codebase.

      Acceptance Criteria

      • Install https://eslint.org/docs/rules/no-shadow into the following repos
        • the driver
        • libmongocrypt node bindings
        • bson
      • Address any new lint failures as a result of this rule.  If shadowing is necessary, add a comment explaining why.
      • Search the codebase for this Jira ticket and address any TODOs (at the time of writing this ticket, there are 3).

            Assignee:
            Unassigned Unassigned
            Reporter:
            bailey.pearson@mongodb.com Bailey Pearson
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: