tooling/ignore-with-reason

Dlaczego

A suppression without an id hides every rule on that line, including problems added later. Without a reason, the next person cannot tell a deliberate exception from a shortcut, and nobody revisits it. The audit cannot report a bare fs-ignore itself, because the comment silences that finding too.

Poprawka

Fix the code when you can. When a finding is a real exception, write the id, then -- and a reason, in the same comment: <!-- fs-ignore a11y/positive-tabindex -- third-party widget sets it, issue #123 -->. Do not put a colon right after the id: a11y/positive-tabindex: is not read as an id, so the comment silences every rule. For a rule the team does not follow at all, put it in audit.ignore in the profile and record why with learn --lesson.

Przykłady

<!-- bad -->
<div tabindex="3"> <!-- fs-ignore -->
<!-- good -->
<!-- fs-ignore a11y/positive-tabindex -- embedded vendor widget, cannot change, issue #123 -->
<div tabindex="3">

Źródła

• Frontend Skills, architecture (suppression)
• ESLint, Disabling rules with inline comments (describe the reason)

Przeczytaj w terminalu

npx github:dawidolko/frontend-skills explain tooling/ignore-with-reason