lucky-parking icon indicating copy to clipboard operation
lucky-parking copied to clipboard

[Tech Debt] Fix ESLint problems with `@typescript-eslint/ban-ts-comment`

Open glenflorendo opened this issue 11 months ago • 0 comments

User Story

As a developer, I want to adhere to our linting rules, so that I can ensure I am writing clean, correct code and following best practices.

Description

ESLint is a popular tool that is used to analyze code and find problems based on our configured rules, which includes several nice recommendations.

This ticket focuses on the @typescript-eslint/ban-ts-comment.

You will need to understand this rule and go through the code to resolve any problems.

To see all violations for all rules, run the following command:

pnpm ci:lint

This rule will continue to persist after this ticket is completed; it should not be removed from our configuration. However, it should be set to error.

Additional Context

This work is being done now in part to Ci/CD efforts. We want to enable linting in our pipeline, however, it will always fail since we currently have errors.

Acceptance Criteria

  1. Running the linter will not find any violations with this rule.
  2. This severity of this rule is set to error.

Technical References

glenflorendo avatar Mar 27 '24 08:03 glenflorendo