Skip to content

Analyzer 8#1292

Closed
lishaduck wants to merge 20 commits intorrousselGit:masterfrom
lishaduck:analyzer-8
Closed

Analyzer 8#1292
lishaduck wants to merge 20 commits intorrousselGit:masterfrom
lishaduck:analyzer-8

Conversation

@lishaduck
Copy link
Copy Markdown
Contributor

@lishaduck lishaduck commented Sep 3, 2025

Depends on #1185.
Depends on invertase/dart_custom_lint#354.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Sep 3, 2025

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing Touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@lishaduck
Copy link
Copy Markdown
Contributor Author

lishaduck commented Sep 4, 2025

Ok, looks like this PR is good except for getting setting correct lower bounds somewhere (and obv. publishing dependent packages)

@rrousselGit
Copy link
Copy Markdown
Owner

Handled separately

@rrousselGit rrousselGit closed this Sep 9, 2025
@lishaduck lishaduck deleted the analyzer-8 branch September 9, 2025 13:34
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