Add eslint rule preventing variable shadowing to eslint configurations

XMLWordPrintableJSON

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

      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
              Reporter:
              Bailey Pearson
              None
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated: