Back to all reviewers

Fix linting root causes

vadimdemedes/ink
Based on 3 comments
TSX

Address underlying code issues rather than disabling linting rules or working around auto-formatter conflicts. When linting tools flag legitimate problems, fix the root cause instead of suppressing the warning. When auto-formatters conflict with intentional code patterns, adjust project-wide configuration rather than using local disables.

Code Style TSX

Reviewer Prompt

Address underlying code issues rather than disabling linting rules or working around auto-formatter conflicts. When linting tools flag legitimate problems, fix the root cause instead of suppressing the warning. When auto-formatters conflict with intentional code patterns, adjust project-wide configuration rather than using local disables.

For example, instead of:

// eslint-disable-next-line @typescript-eslint/ban-ts-comment
// @ts-ignore
internal_static

Fix the underlying TypeScript issue that requires the ignore comment.

When auto-formatters incorrectly modify intentional test patterns like A{'B'} to AB, configure the formatter globally rather than disabling rules on individual lines. This maintains code quality while preserving intentional code structure.

3
Comments Analyzed
TSX
Primary Language
Code Style
Category

Source Discussions