Enable typescript-eslint/no-floating-promises eslint rule

XMLWordPrintableJSON

    • Type: Task
    • Resolution: Done
    • Priority: Critical - P2
    • None
    • Affects Version/s: None
    • Component/s: None
    • 2
    • None
    • Not Needed
    • None
    • None
    • None
    • None
    • None
    • None

      Use Case

      As a node engineer
      I want my linter to catch issues with promises not having rejection handling
      So that I don't write floating promise code

      User Impact

      *What is the number of impacted customers? How severe is the impact? Is anyone blocked or broken?

      Dependencies

      • upstream and/or downstream requirements and timelines to bear in mind

      Unknowns

      • questions that need to be answered to determine implementation

      Acceptance Criteria

          {
            // Settings for typescript src files
            "files": [
              "src/**/*.ts"
            ],
            "parser": "@typescript-eslint/parser",
            "parserOptions": {
              "project": ["./tsconfig.json"]
            },
            "extends": [
              "plugin:@typescript-eslint/recommended-requiring-type-checking"
            ],
            "rules": {
              "@typescript-eslint/no-unsafe-member-access": "off",
              "@typescript-eslint/no-unsafe-argument": "off",
              "@typescript-eslint/no-unsafe-assignment": "off",
              "@typescript-eslint/no-unsafe-return": "off",
              "@typescript-eslint/no-unsafe-call": "off",
      
              "@typescript-eslint/restrict-plus-operands": "off",
              "@typescript-eslint/restrict-template-expressions": "off"
            }
          },
      

      Implementation Requirements

      • Change these lint rules in the driver
      • Change these lint rules in the BSON library

      Testing Requirements

      • unit test, spec test sync, etc

      Documentation Requirements

      • DOCSP ticket, API docs, etc

      Follow Up Requirements

      • additional tickets to file, required releases, etc

            Assignee:
            Neal Beeken
            Reporter:
            Neal Beeken
            Bailey Pearson
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: