Sitelet https://github.com/flutter/flutter/pull/191071
Skip to content

ignore soon-to-be-removed plugins_in_inner_options issue - #191071

Merged
srawlins merged 1 commit into
flutter:masterfrom
srawlins:ignore-inner
Aug 13, 2026
Merged

srawlins merged 1 commit into
flutter:masterfrom
srawlins:ignore-inner

Conversation

@srawlins

@srawlins srawlins commented Aug 13, 2026 •

Copy link
Copy Markdown
Contributor

@bkonyi landed #191022 around the same time I landed https://dart-review.googlesource.com/c/sdk/+/534500. They sort of conflict in that Ben's PR references plugins_in_inner_options which my CL deletes. So we can ignore the unrecognized_error_code until a roll makes it in.

See https://logs.chromium.org/logs/dart/buildbucket/cr-buildbucket/8673632836896349697/+/u/analyze_Flutter_repositories/stdout which I believe is from the HH bot.

@flutter-dashboard flutter-dashboard Bot added the CICD Run CI/CD label Aug 13, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds a commented-out inline ignore directive to analysis_options.yaml. Feedback indicates that the Dart analyzer does not support inline ignore comments within YAML configuration files, making the added directive ineffective for suppressing warnings.

Comment thread analysis_options.yaml
Comment on lines +11 to +12
# This error code will no longer be reported here. Until dart rolls in, we need to just ignore the issue.
# ignore: unrecognized_error_code

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The Dart analyzer does not support inline ignore comments (such as # ignore: unrecognized_error_code) within YAML configuration files like analysis_options.yaml. Inline ignores are only supported in Dart source files (.dart). As a result, this comment is ineffective and will not suppress the warning when the error code is removed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CICD Run CI/CD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants