Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[flutter_lints] remove prefer_const_constructors, prefer_const_declarations, prefer_const_literals_to_create_immutables #7688

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

devoncarew
Copy link
Member

This PR removes three const lints from package:flutter_lints: prefer_const_constructors, prefer_const_declarations, and prefer_const_literals_to_create_immutables.

This PR does not rev the pubspec version. We want to stage this change out in coordination with package:lints; see dart-lang/lints#209.

Pre-launch Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@devoncarew
Copy link
Member Author

I believe this PR is except from the changelog check, as we're intentionally staging this change in prep for a publish once the next major version of package:lints is available.

@devoncarew devoncarew added the override: no versioning needed Override the check requiring version bumps for most changes label Sep 23, 2024
@devoncarew
Copy link
Member Author

cc @Piinks and @stuartmorgan as likely reviewers (normally I would ask @goderbauer for a review here / he would author the PR).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
override: no versioning needed Override the check requiring version bumps for most changes p: flutter_lints
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant