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

migrate the analysis config to package:dart_flutter_team_lints #2511

Merged
merged 5 commits into from
May 4, 2023

Conversation

devoncarew
Copy link
Member

  • migrate the analysis config to package:dart_flutter_team_lints

This PR doesn't contain any semantic changes (but does delete some dead code).

@devoncarew
Copy link
Member Author

Hmm, interesting - the stable channel has analysis issues; the beta channel doesn't, but fails two tests (though is currently running with continue-on-error=true).

@devoncarew devoncarew requested a review from johnpryan May 4, 2023 21:29
lib/sharing/gists.dart Outdated Show resolved Hide resolved
@@ -54,7 +54,7 @@ serveLocalAppEngine() async {
_serverUrlOption: 'http://127.0.0.1:8084/',
}),
))
serveLocalBackend() async {
Future serveLocalBackend() async {
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a lint rule for avoiding incomplete generic types?

Copy link
Member Author

Choose a reason for hiding this comment

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

Possibly? Or perhaps this would be caught by enabling one of:

analyzer:
  language:
    strict-inference: true
    strict-raw-types: true

Co-authored-by: John Ryan <ryjohn@google.com>
@devoncarew devoncarew merged commit 47ca147 into master May 4, 2023
3 checks passed
@devoncarew devoncarew deleted the dart_flutter_team_lints branch May 4, 2023 23:08
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.

None yet

2 participants