Skip to content

Conversation

davidmorgan
Copy link
Contributor

Analyzer 8 won't version solve yet due to build+dart_style, I tested it with a dep override here

#777

and added ignores + one tiny fix, copied here.

So this should be good to go on 7+8 now :)

@davidmorgan davidmorgan requested a review from kevmoo August 7, 2025 11:19
@davidmorgan davidmorgan marked this pull request as ready for review August 7, 2025 11:19
@davidmorgan davidmorgan mentioned this pull request Aug 7, 2025
Copy link
Member

@kevmoo kevmoo left a comment

Choose a reason for hiding this comment

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

Please update the example constraint, too!

@kevmoo
Copy link
Member

kevmoo commented Aug 7, 2025

diff --git a/example/pubspec.yaml b/example/pubspec.yaml
index 8cda9aa..96ef2a3 100644
--- a/example/pubspec.yaml
+++ b/example/pubspec.yaml
@@ -6,8 +6,8 @@ environment:
   sdk: ^3.7.0
 
 dependencies:
-  analyzer: ^7.4.0
-  build: ^3.0.0-dev
+  analyzer: '>=7.4.0 <9.0.0'
+  build: ^3.0.0
   source_gen: any
 
 dev_dependencies:

Copy link
Member

@kevmoo kevmoo left a comment

Choose a reason for hiding this comment

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

I gottcha!

@davidmorgan
Copy link
Contributor Author

Thanks :)

@davidmorgan davidmorgan merged commit b864909 into dart-lang:master Aug 8, 2025
12 checks passed
@davidmorgan davidmorgan deleted the release-3-1-0 branch August 8, 2025 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants