Skip to content

Commit 87079f4

Browse files
xsahil03xclaude
andcommitted
chore(repo): enable migrate_design_widgets to guard the migration
Without the lint nothing stops a new file importing package:flutter/material.dart, silently reintroducing the split between Flutter's Material types and material_ui's. Enabling it is safe at the 3.44 floor even though that analyzer does not recognise the rule. `undefined_lint` is reported against analysis_options.yaml itself, so it only surfaces when that file is inside the analysed directory. legacy_version_analyze runs `cd packages/X/lib && dart analyze --fatal-infos .`, which leaves the options file out of scope: verified exit 0 from lib/ versus exit 2 from the package root with the same unknown rule enabled. The main analyze job runs the latest stable, which knows the rule. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
1 parent a5b2a15 commit 87079f4

1 file changed

Lines changed: 6 additions & 1 deletion

File tree

analysis_options.yaml

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -154,4 +154,9 @@ linter:
154154
- unnecessary_overrides
155155
- prefer_null_aware_method_calls
156156
- use_named_constants
157-
- use_raw_strings
157+
- use_raw_strings
158+
# Guards the material_ui/cupertino_ui migration: fails any new import
159+
# of package:flutter/material.dart or package:flutter/cupertino.dart.
160+
# Unknown to the 3.44 floor, but legacy_version_analyze analyses lib/
161+
# rather than the package root, so it never sees this file.
162+
- migrate_design_widgets

0 commit comments

Comments
 (0)