Skip to content
This repository has been archived by the owner on Jun 20, 2024. It is now read-only.

Enable and fix package:pedantic lints #135

Merged
merged 2 commits into from
Jan 13, 2020
Merged

Conversation

natebosch
Copy link
Contributor

  • always_declare_return_types
  • omit_local_variable_types
  • prefer_collection_literals
  • prefer_if_null_operators
  • unawaited_futures

Bump min SDK to 2.2.0 to support Set literals.

- always_declare_return_types
- omit_local_variable_types
- prefer_collection_literals
- prefer_if_null_operators
- unawaited_futures

Bump min SDK to 2.2.0 to support Set literals.
@googlebot googlebot added the cla: yes Google CLA signed label Dec 9, 2019
@natebosch natebosch requested a review from grouma December 9, 2019 21:39
@devoncarew
Copy link
Contributor

Thanks for the PRs @natebosch; @grouma, @natebosch, I'd like the opportunity to review these before they get merged.

@natebosch natebosch requested review from devoncarew and removed request for grouma December 9, 2019 21:50
@natebosch
Copy link
Contributor Author

@devoncarew - any update?

@kevmoo
Copy link
Contributor

kevmoo commented Jan 13, 2020

LGTM! @devoncarew ?

Copy link
Contributor

@devoncarew devoncarew left a comment

Choose a reason for hiding this comment

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

As per the recent design doc from mit, pedantic is too pedantic. I'll update this package to use whatever LCD rule set we come up with once it exists.

@@ -1,3 +1,4 @@
include: package:pedantic/analysis_options.yaml
Copy link
Contributor

Choose a reason for hiding this comment

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

We should peg this to a specific version so that changes to the pedantic package won't break the CI here.

@natebosch
Copy link
Contributor Author

pedantic is too pedantic

FWIW omit_local_variable_types is the only lint that showed up here that might be excluded from other attempts at a standardized lint set. The others are very likely to be included.

@natebosch natebosch merged commit c2051dd into master Jan 13, 2020
@natebosch natebosch deleted the package-pedantic-lints branch January 13, 2020 21:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes Google CLA signed
Development

Successfully merging this pull request may close these issues.

4 participants