From 701cd77844840fe3cbedcefadef601a065df661f Mon Sep 17 00:00:00 2001 From: Paul Berry Date: Thu, 21 Mar 2024 16:38:11 +0000 Subject: [PATCH] Ignore TODO in pkg/analyzer_plugin. The analyzer team has decided to adopt the convention of using `TODO` comments to document long term issues that should persist in the codebase, and `FIXME` comments to document short term issues that need immediate attention. They may even consider adding a presubmit hook to ensure that `FIXME` comments are only used during local development. Accordingly, it makes sense to suppress `TODO` comments from being surfaced to the IDE "problems" view (since there are hundreds of them, and they're not immediately actionable). This makes VSCode's "problems" view much more usable in "tree" mode. (See also https://dart-review.googlesource.com/c/sdk/+/295662, which made the corresponding change to the `analysis_server` and `analyzer` packages, and https://dart-review.googlesource.com/c/sdk/+/325121, which made the corresponding change to the `linter` package). Change-Id: Ifbd2f6bed468408b1906b25ab94f745eed0b3e55 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/358980 Reviewed-by: Konstantin Shcheglov Commit-Queue: Paul Berry --- pkg/analyzer_plugin/analysis_options.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/pkg/analyzer_plugin/analysis_options.yaml b/pkg/analyzer_plugin/analysis_options.yaml index 7ba7802f9b2..2565e538aff 100644 --- a/pkg/analyzer_plugin/analysis_options.yaml +++ b/pkg/analyzer_plugin/analysis_options.yaml @@ -4,6 +4,7 @@ analyzer: language: strict-casts: true errors: + todo: ignore # Remove from lints/recommended: constant_identifier_names: ignore implementation_imports: ignore