From 3e195c913f4039240f4bf83f069c68fea63ec02e Mon Sep 17 00:00:00 2001 From: Paul Berry Date: Tue, 26 May 2026 10:40:55 -0700 Subject: [PATCH] Ignore TODOs in analysis_server_client package. Some time ago, the analyzer team 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. Accordingly, it makes sense to suppress `TODO` comments from being surfaced to the IDE "problems" view (since there can be a lot of them, and they're not immediately actionable). This makes VSCode's "problems" view much more usable in "tree" mode. Previous CLs that have made this change in other packages: - https://dart-review.googlesource.com/c/sdk/+/295662 - https://dart-review.googlesource.com/c/sdk/+/325121 - https://dart-review.googlesource.com/c/sdk/+/358980 - https://dart-review.googlesource.com/c/sdk/+/410060 - https://dart-review.googlesource.com/c/sdk/+/440546 - https://dart-review.googlesource.com/c/sdk/+/442167 - https://dart-review.googlesource.com/c/sdk/+/443633 Change-Id: Ia7c7ce5e0ab0ac32d61d8a93319c363c6a6a6964 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/506321 Commit-Queue: Paul Berry Reviewed-by: Brian Wilkerson --- pkg/analysis_server_client/analysis_options.yaml | 3 +++ 1 file changed, 3 insertions(+) diff --git a/pkg/analysis_server_client/analysis_options.yaml b/pkg/analysis_server_client/analysis_options.yaml index 323633d7927..65b34cd2f63 100644 --- a/pkg/analysis_server_client/analysis_options.yaml +++ b/pkg/analysis_server_client/analysis_options.yaml @@ -4,6 +4,9 @@ analyzer: errors: constant_identifier_names: ignore non_constant_identifier_names: ignore + # Ignoring this for all developers means developers don't need to ignore + # TODOs in their IDE settings. + todo: ignore # Existing violations (109) unintended_html_in_doc_comment: ignore