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: