Skip to content

Commit 197bbbd

Browse files
stereotype441Commit Queue
authored andcommitted
Ignore TODOs in analysis_server_plugin.
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. (See also https://dart-review.googlesource.com/c/sdk/+/295662, which made the corresponding change to the `analysis_server` and `analyzer` packages, https://dart-review.googlesource.com/c/sdk/+/325121, which made the corresponding change to the `linter` package, and https://dart-review.googlesource.com/c/sdk/+/358980, which made the corresponding change to the `analyzer_plugin` package). Change-Id: I916c8c7f435d06c1b0bd871ef3829d58c9242218 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/410060 Reviewed-by: Samuel Rawlins <[email protected]> Auto-Submit: Paul Berry <[email protected]> Commit-Queue: Paul Berry <[email protected]>
1 parent 7964d55 commit 197bbbd

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

pkg/analysis_server_plugin/analysis_options.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,10 @@ analyzer:
1212
implementation_imports: ignore
1313
non_constant_identifier_names: ignore
1414

15+
# Ignoring this for all developers means developers don't need to ignore
16+
# TODOs in their IDE settings.
17+
todo: ignore
18+
1519
linter:
1620
rules:
1721
- always_use_package_imports

0 commit comments

Comments
 (0)