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

[analysis] Clean up legacy custom rules and private-lints in analyze.dart - #191128

Merged
auto-submit[bot] merged 4 commits into
flutter:masterfrom
bkonyi:cleanup_legacy_custom_rules
Aug 16, 2026
Merged

auto-submit[bot] merged 4 commits into
flutter:masterfrom
bkonyi:cleanup_legacy_custom_rules

Conversation

@bkonyi

@bkonyi bkonyi commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Removes the legacy dev/bots/custom_rules/ folder and private-lints validation in dev/bots/analyze.dart now that all rules have been migrated to dev/flutter_analyzer_plugin.

Changes

  • Deletes dev/bots/custom_rules/ directory and all legacy rule implementations.
  • Removes _verifyPrivateLints and 'private-lints' from dev/bots/analyze.dart.
  • Removes obsolete tests from dev/bots/test/analyze_test.dart.
  • Updates documentation reference in packages/flutter/lib/src/widgets/framework.dart.

@flutter-dashboard flutter-dashboard Bot added the CICD Run CI/CD label Aug 14, 2026
@github-actions github-actions Bot added the framework flutter/packages/flutter repository. See also f: labels. label Aug 14, 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 removes custom private lints and their associated tests from the dev/bots directory, including rules for double.clamp, Stopwatch, RenderBox intrinsics, and public State subtypes. Additionally, it updates a reference comment in packages/flutter/lib/src/widgets/framework.dart to point to the new location of these rules under dev/flutter_analyzer_plugin. There are no review comments, and I have no feedback to provide.

@github-actions github-actions Bot added the p: material_ui material_ui package in flutter/packages label Aug 14, 2026
@github-actions github-actions Bot removed the p: material_ui material_ui package in flutter/packages label Aug 14, 2026
@bkonyi
bkonyi requested a review from srawlins August 15, 2026 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CICD Run CI/CD framework flutter/packages/flutter repository. See also f: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants