Skip to content

handle prefer_equal_for_default_values deprecation in 2.19 #786

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

Closed
pq opened this issue Nov 29, 2022 · 5 comments
Closed

handle prefer_equal_for_default_values deprecation in 2.19 #786

pq opened this issue Nov 29, 2022 · 5 comments
Labels
package:lints type-question A question about expected behavior or functionality

Comments

@pq
Copy link
Member

pq commented Nov 29, 2022

In 2.19.0 (as of dart-lang/sdk@289aa50), analyzer produces a warning when colons are used for default values. This makes prefer_equal_for_default_values redundant and slated for deprecation (dart-archive/linter#3855).

I plan to cherry-pick a linter that doesn't produce a lint for SDKs >= 2.19.0-0 so we won't double-report but we should still remove this lint from the recommended set. Is it worth a package:lints release that only removes this lint for SDKs >= 2.19.0-0? Other considerations?

/cc @bwilkerson @mit-mit @munificent @natebosch @jacob314 @jakemac53 @lrhn @devoncarew @goderbauer

@pq pq added the type-question A question about expected behavior or functionality label Nov 29, 2022
@natebosch
Copy link
Member

Seems fine to release.

It might also be a good reason to evaluate if there are new lints we should include. implicit_call_tearoffs is worth including right away.

@bwilkerson
Copy link
Member

The one consideration I can think of is that if we release a new version of the lint set, and a user moves to the new version without also moving to 2.19, they will lose coverage.

@jakemac53
Copy link
Contributor

Generally SGTM, if we do increase the min sdk to 2.19 we should just keep in mind that means users on older SDKs won't get any new lint sets. If we don't increase it then as brian says we lose some coverage on the lint.

I would probably suggest moving to a min sdk of 2.19, but I don't know when stable of that is slated for release. If it isn't soonish then we might want to hold off, or make sure we do an earlier release with a last set of changes to the lints that we want people to get prior to 2.19 stable.

@mit-mit
Copy link
Member

mit-mit commented Nov 30, 2022

don't know when stable of that is slated for release

Roughly end of January 2023, is the current plan

@devoncarew
Copy link
Member

@pq - is this complete?

@pq pq closed this as completed Jun 1, 2023
@mosuem mosuem transferred this issue from dart-archive/lints Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package:lints type-question A question about expected behavior or functionality
Projects
None yet
Development

No branches or pull requests

7 participants