Skip to content
This repository was archived by the owner on Jan 14, 2025. It is now read-only.

Fix new type error, use dart_flutter_team_lints #80

Merged
merged 1 commit into from
Feb 13, 2023
Merged

Conversation

kevmoo
Copy link
Contributor

@kevmoo kevmoo commented Feb 12, 2023

No description provided.

@kevmoo kevmoo requested a review from devoncarew February 12, 2023 21:26
@kevmoo
Copy link
Contributor Author

kevmoo commented Feb 12, 2023

Feel free to squash/merge and delete the branch if you're happy

@@ -1,5 +1,5 @@
# https://dart.dev/guides/language/analysis-options
include: package:lints/recommended.yaml
include: package:dart_flutter_team_lints/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.

low: technically you can also remove the strict-casts: true below (but not sure it's worth an update here to do that; we should just work towards being able to use dart_flutter_team_lints w/o many or any modifications).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I need to add that logic to my checker tool in

@kevmoo kevmoo merged commit 0d62000 into master Feb 13, 2023
@kevmoo kevmoo deleted the lint_ci_fix branch February 13, 2023 01:07
mosuem pushed a commit to dart-lang/tools that referenced this pull request Dec 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants