[L] Change in dart/sdk[main]: Parse and validate analyzer plugins in "inner" options files
0 views
Skip to first unread message
Brian Wilkerson (Gerrit)
Jul 15, 2026, 5:42:47 PM (10 hours ago) Jul 15
to Samuel Rawlins, Brian Wilkerson, dart-...@luci-project-accounts.iam.gserviceaccount.com, dart-analys...@google.com, rev...@dartlang.org
Attention needed from Samuel Rawlins
Brian Wilkerson voted Code-Review+1![]( "Open in Gerrit")
| Code-Review | +1 |
Related details
Attention is currently required from:
- Samuel Rawlins
Submit Requirements:
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Samuel Rawlins (Gerrit)
Jul 15, 2026, 6:16:49 PM (10 hours ago) Jul 15
to Brian Wilkerson, dart-...@luci-project-accounts.iam.gserviceaccount.com, dart-analys...@google.com, rev...@dartlang.org
Samuel Rawlins voted Commit-Queue+2![]( "Open in Gerrit")
Submit Requirements:
Code-Owners
Code-Review
Review-Enforcement
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
dart-scoped@luci-project-accounts.iam.gserviceaccount.com (Gerrit)
Jul 15, 2026, 6:17:00 PM (10 hours ago) Jul 15
to Samuel Rawlins, Brian Wilkerson, dart-analys...@google.com, rev...@dartlang.org
dart-...@luci-project-accounts.iam.gserviceaccount.com submitted the change![]( "Open in Gerrit")
Change information
Commit message:
Parse and validate analyzer plugins in "inner" options files
Work towards https://github.com/dart-lang/sdk/issues/63627
We don't yet act on the inner options files. That is next. But there
is enough code and testing just in validating that they don't conflict,
so this CL is first.
Change-Id: I4180018cb536e0d996e50395dc2611e65d799bac
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/524521
Reviewed-by: Brian Wilkerson <brianwi...@google.com>
Commit-Queue: Samuel Rawlins <sraw...@google.com>
Files:
- M pkg/analysis_server/lib/src/context_manager.dart
- M pkg/analysis_server/lib/src/services/correction/error_fix_status.yaml
- M pkg/analysis_server/pubspec.yaml
- M pkg/analysis_server/test/analysis/notification_analysis_options_test.dart
- M pkg/analysis_server/test/domain_analysis_test.dart
- M pkg/analyzer/lib/src/diagnostic/diagnostic.g.dart
- M pkg/analyzer/lib/src/diagnostic/diagnostic_code_values.g.dart
- M pkg/analyzer/messages.yaml
Change size: L
Delta: 8 files changed, 423 insertions(+), 41 deletions(-)
Branch: refs/heads/main
Submit Requirements:
Code-Review: +1 by Brian Wilkerson
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Reply all
Reply to author
Forward
0 new messages