Skip to content

feat: add unnecessary_primary_constructor_body rule - #234

Merged
marcossevilla merged 3 commits into
mainfrom
vgv-ai-bot/issue-217
Aug 17, 2026
Merged

feat: add unnecessary_primary_constructor_body rule#234
marcossevilla merged 3 commits into
mainfrom
vgv-ai-bot/issue-217

Conversation

@unicoderbot

@unicoderbot unicoderbot Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Summary

Enables the unnecessary_primary_constructor_body linter rule introduced in Dart 3.13, which detects unnecessary bodies in primary constructors.

Changes

  • Added unnecessary_primary_constructor_body to analysis_options.11.0.0.yaml
  • Removed the rule from the exclusion list in exclusion_reasons.json and README.md

Closes #217

Closes #217

Co-authored-by: marcossevilla <marcossevilla@users.noreply.github.com>
marcossevilla
marcossevilla previously approved these changes Aug 17, 2026
@marcossevilla

Copy link
Copy Markdown
Member

@vgvbot fix CI

@unicoderbot

unicoderbot Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor Author

On it, I'll address the review comments and push the changes.

  • Preparing environment
    • Cloning repository
    • Setting up dev environment
  • Reviewing feedback
  • Working on changes
  • Pushing updates

I've pushed changes to address the feedback.

Co-authored-by: marcossevilla <marcossevilla@users.noreply.github.com>
@marcossevilla
marcossevilla merged commit 53e3f8b into main Aug 17, 2026
6 checks passed
@marcossevilla
marcossevilla deleted the vgv-ai-bot/issue-217 branch August 17, 2026 11:42
@vgvbot vgvbot mentioned this pull request Aug 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

feat: add unnecessary_primary_constructor_body rule

1 participant