Skip to content
This repository has been archived by the owner on Dec 1, 2024. It is now read-only.

DontSuppressLintErrorsLinter #543

Open
lexidor opened this issue May 24, 2023 · 0 comments
Open

DontSuppressLintErrorsLinter #543

lexidor opened this issue May 24, 2023 · 0 comments

Comments

@lexidor
Copy link
Contributor

lexidor commented May 24, 2023

The idea is described in this issue: #458.

This ought be a be linter, instead of a framework feature.

This linter should take a configuration option with a deny_list of linter you may not suppress and/or an allow_list for linters you should be able to suppress. If neither is provided, do nothing. This allows users to upgrade to a later version of hhast without all their suppression comments being ignored.

Optional, but highly recommended, make it impossible to suppress DontSuppressLintErrorsLinter. This is kinda silly.

/** HHAST_IGNORE_ERROR[DontSuppressLintErrors] I know you to sneak this by you. */
/** HHAST_IGNORE_ERROR[DontUseAsioJoin] not in the allow_list. */
Asio\join($awaitable);
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant