-
Notifications
You must be signed in to change notification settings - Fork 170
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
add missing lints in analysis_options.yaml #1069
Conversation
Thinking out loud: should we consider adding a validation step to the tests to ensure this stays up to date? See also #1051, which has I think some relevant conversation. |
analysis_options.yaml
Outdated
@@ -16,20 +16,30 @@ linter: | |||
- annotate_overrides | |||
- avoid_annotating_with_dynamic | |||
# - avoid_as | |||
# - avoid_bool_literals_in_conditional_expressions # not yet tested |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, on reading this again, I'm not sure this note is totally accurate. Maybe a link to #1068 instead? Maybe something like:
# under review (see #1068)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On second thought, would you consider updating the comment text?
PTAL |
Super. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
* master: (41 commits) New lint: prefer_void_to_null (dart-lang#1100) Fix warnings in the linter code base echo dart version (dart-lang#1103) Update google.yaml (dart-lang#1094) improved description (dart-lang#1092) Address review comments special case with ignore for lines_longer_than_80_chars 0.1.58 (dart-lang#1086) Roll-back use of optional new/const; add `no-preview-dart-2` bot. (dart-lang#1084) 0.1.57 (dart-lang#1080) address review comments address review comments fix lines_longer_than_80_chars for \r Fix npe in non_constant_identifier_names chore: set max SDK version to <3.0.0 (dart-lang#1077) Update google.yaml (dart-lang#1076) Google internal rules (dart-lang#1075) Prevent an NPE when dealing with incomplete/invalid code add missing lints in analysis_options.yaml (dart-lang#1069) fomatting fixes (dart-lang#1067) ...
No description provided.