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

[analysis] Clarify plugin relative path resolution in root analysis_options.yaml - #191077

Merged
srawlins merged 2 commits into
flutter:masterfrom
bkonyi:update_analysis_options_comment
Aug 13, 2026
Merged

srawlins merged 2 commits into
flutter:masterfrom
bkonyi:update_analysis_options_comment

Conversation

@bkonyi

@bkonyi bkonyi commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Description

Updates the comment in root analysis_options.yaml to accurately explain that analyzer plugin paths are resolved relative to the subpackage directory evaluating the options, so relative plugin paths cannot be inherited from root options.

Related Issues

N/A

Tests

  • Verified with bin/flutter analyze --flutter-repo

@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 updates the explanatory comment in analysis_options.yaml to clarify that Dart analyzer plugins must be configured directly within each subpackage's configuration file rather than at the repository root, as relative plugin paths cannot be inherited. There are no review comments, and I have no feedback to provide.

@bkonyi
bkonyi requested a review from srawlins August 13, 2026 16:08
@bkonyi bkonyi added the autosubmit Merge PR when tree becomes green via auto submit App label Aug 13, 2026
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