Skip to content

Commit 11ccf79

Browse files
committed
chore: update rules and exclusion table
1 parent b644968 commit 11ccf79

File tree

13 files changed

+254
-18
lines changed

13 files changed

+254
-18
lines changed

.github/workflows/bot_updater.yaml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ jobs:
2323
- name: 🎯 Setup Dart
2424
uses: dart-lang/setup-dart@v1
2525
with:
26-
sdk: "3.9.0"
26+
sdk: "3.10.0"
2727

2828
- name: 📦 Install Dependencies
2929
run: dart pub get
@@ -56,6 +56,8 @@ jobs:
5656
author: VGV Bot <vgvbot@users.noreply.github.com>
5757
assignees: vgvbot
5858
committer: VGV Bot <vgvbot@users.noreply.github.com>
59+
token: ${{ steps.generate-token.outputs.token }}
60+
sign-commits: true
5961

6062
- name: 🔍 Check for deprecated rules changes
6163
id: deprecated
@@ -79,3 +81,5 @@ jobs:
7981
author: VGV Bot <vgvbot@users.noreply.github.com>
8082
assignees: vgvbot
8183
committer: VGV Bot <vgvbot@users.noreply.github.com>
84+
token: ${{ steps.generate-token.outputs.token }}
85+
sign-commits: true

.github/workflows/main.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ jobs:
1212
- uses: actions/checkout@v5
1313
- uses: dart-lang/setup-dart@v1
1414
with:
15-
sdk: "3.9.0"
15+
sdk: "3.10.0"
1616

1717
- name: Install Dependencies
1818
run: dart pub get

.github/workflows/pub_publish.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ jobs:
1616
- name: 🎯 Setup Dart
1717
uses: dart-lang/setup-dart@v1
1818
with:
19-
sdk: "3.9.0"
19+
sdk: "3.10.0"
2020
- name: 📦 Install Dependencies
2121
run: dart pub get
2222
- name: 🌵 Dry Run

.github/workflows/tool_linter_rules.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,5 +15,5 @@ jobs:
1515
build:
1616
uses: VeryGoodOpenSource/very_good_workflows/.github/workflows/dart_package.yml@v1
1717
with:
18-
dart_sdk: 3.9.0
18+
dart_sdk: 3.10.0
1919
working_directory: tool/linter_rules

.github/workflows/verify_version.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ jobs:
1414

1515
- uses: dart-lang/setup-dart@v1
1616
with:
17-
sdk: "3.9.0"
17+
sdk: "3.10.0"
1818

1919
- name: Verify version consistency
2020
run: dart run tool/verify_version/main.dart

README.md

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ Below is a list of rules that are not enabled by default together with the reaso
105105
| [`diagnostic_describe_all_properties`](https://dart.dev/tools/linter-rules/diagnostic_describe_all_properties) | Not specified |
106106
| [`do_not_use_environment`](https://dart.dev/tools/linter-rules/do_not_use_environment) | Not specified |
107107
| [`library_names`](https://dart.dev/tools/linter-rules/library_names) | [Superseded by `unnecessary_library_name`](https://github.com/dart-lang/lints/issues/172) |
108-
| [`omit_obvious_local_variable_types`](https://dart.dev/tools/linter-rules/omit_obvious_local_variable_types) | Not specified |
108+
| [`omit_obvious_local_variable_types`](https://dart.dev/tools/linter-rules/omit_obvious_local_variable_types) | Experimental |
109109
| [`omit_obvious_property_types`](https://dart.dev/tools/linter-rules/omit_obvious_property_types) | Incompatible with [type_annotate_public_apis](https://github.com/dart-lang/sdk/issues/60642) |
110110
| [`package_prefixed_library_names`](https://dart.dev/tools/linter-rules/package_prefixed_library_names) | [Broken; superseded by `unnecessary_library_name`](https://github.com/dart-lang/lints/issues/172) |
111111
| [`prefer_double_quotes`](https://dart.dev/tools/linter-rules/prefer_double_quotes) | Incompatible with [prefer_single_quotes](https://dart.dev/tools/linter-rules/prefer_single_quotes) |
@@ -114,10 +114,11 @@ Below is a list of rules that are not enabled by default together with the reaso
114114
| [`prefer_mixin`](https://dart.dev/tools/linter-rules/prefer_mixin) | Not specified |
115115
| [`prefer_relative_imports`](https://dart.dev/tools/linter-rules/prefer_relative_imports) | Incompatible with [always_use_package_imports](https://dart.dev/tools/linter-rules/always_use_package_imports) |
116116
| [`prefer_void_to_null`](https://dart.dev/tools/linter-rules/prefer_void_to_null) | [Has unresolved false positives](https://github.com/dart-lang/linter/issues/4758) |
117-
| [`specify_nonobvious_local_variable_types`](https://dart.dev/tools/linter-rules/specify_nonobvious_local_variable_types) | Not specified |
118-
| [`unnecessary_async`](https://dart.dev/tools/linter-rules/unnecessary_async) | Not specified |
117+
| [`remove_deprecations_in_breaking_versions`](https://dart.dev/tools/linter-rules/remove_deprecations_in_breaking_versions) | Not specified |
118+
| [`specify_nonobvious_local_variable_types`](https://dart.dev/tools/linter-rules/specify_nonobvious_local_variable_types) | Experimental |
119+
| [`unnecessary_async`](https://dart.dev/tools/linter-rules/unnecessary_async) | Experimental |
119120
| [`unnecessary_final`](https://dart.dev/tools/linter-rules/unnecessary_final) | Incompatible with [prefer_final_locals](https://dart.dev/tools/linter-rules/prefer_final_locals) |
120-
| [`unsafe_variance`](https://dart.dev/tools/linter-rules/unsafe_variance) | Not specified |
121+
| [`unsafe_variance`](https://dart.dev/tools/linter-rules/unsafe_variance) | Experimental |
121122
| [`use_decorated_box`](https://dart.dev/tools/linter-rules/use_decorated_box) | [Has unresolved malfunctions](https://github.com/dart-lang/linter/issues/3286) |
122123
<!-- end:excluded_rules_table -->
123124

example/pubspec.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ version: 0.0.1
33
publish_to: none
44

55
environment:
6-
sdk: ^3.9.0
6+
sdk: ^3.10.0
77

88
dependencies:
99
very_good_analysis:

lib/analysis_options.10.1.0.yaml

Lines changed: 230 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,230 @@
1+
analyzer:
2+
language:
3+
strict-casts: true
4+
strict-inference: true
5+
strict-raw-types: true
6+
7+
errors:
8+
close_sinks: ignore
9+
missing_required_param: error
10+
missing_return: error
11+
record_literal_one_positional_no_trailing_comma: error
12+
collection_methods_unrelated_type: warning
13+
unrelated_type_equality_checks: warning
14+
15+
exclude:
16+
- test/.test_coverage.dart
17+
- lib/generated_plugin_registrant.dart
18+
19+
formatter:
20+
trailing_commas: preserve
21+
22+
linter:
23+
rules:
24+
- always_declare_return_types
25+
- always_put_required_named_parameters_first
26+
- always_use_package_imports
27+
- annotate_overrides
28+
- avoid_bool_literals_in_conditional_expressions
29+
- avoid_catches_without_on_clauses
30+
- avoid_catching_errors
31+
- avoid_double_and_int_checks
32+
- avoid_dynamic_calls
33+
- avoid_empty_else
34+
- avoid_equals_and_hash_code_on_mutable_classes
35+
- avoid_escaping_inner_quotes
36+
- avoid_field_initializers_in_const_classes
37+
- avoid_final_parameters
38+
- avoid_function_literals_in_foreach_calls
39+
- avoid_init_to_null
40+
- avoid_js_rounded_ints
41+
- avoid_multiple_declarations_per_line
42+
- avoid_positional_boolean_parameters
43+
- avoid_print
44+
- avoid_private_typedef_functions
45+
- avoid_redundant_argument_values
46+
- avoid_relative_lib_imports
47+
- avoid_renaming_method_parameters
48+
- avoid_return_types_on_setters
49+
- avoid_returning_null_for_void
50+
- avoid_returning_this
51+
- avoid_setters_without_getters
52+
- avoid_shadowing_type_parameters
53+
- avoid_single_cascade_in_expression_statements
54+
- avoid_slow_async_io
55+
- avoid_type_to_string
56+
- avoid_types_as_parameter_names
57+
- avoid_unnecessary_containers
58+
- avoid_unused_constructor_parameters
59+
- avoid_void_async
60+
- avoid_web_libraries_in_flutter
61+
- await_only_futures
62+
- camel_case_extensions
63+
- camel_case_types
64+
- cancel_subscriptions
65+
- cascade_invocations
66+
- cast_nullable_to_non_nullable
67+
- collection_methods_unrelated_type
68+
- combinators_ordering
69+
- comment_references
70+
- conditional_uri_does_not_exist
71+
- constant_identifier_names
72+
- control_flow_in_finally
73+
- curly_braces_in_flow_control_structures
74+
- dangling_library_doc_comments
75+
- depend_on_referenced_packages
76+
- deprecated_consistency
77+
- deprecated_member_use_from_same_package
78+
- directives_ordering
79+
- discarded_futures
80+
- document_ignores
81+
- empty_catches
82+
- empty_constructor_bodies
83+
- empty_statements
84+
- eol_at_end_of_file
85+
- exhaustive_cases
86+
- file_names
87+
- flutter_style_todos
88+
- hash_and_equals
89+
- implicit_call_tearoffs
90+
- implementation_imports
91+
- implicit_reopen
92+
- invalid_case_patterns
93+
- invalid_runtime_check_with_js_interop_types
94+
- join_return_with_assignment
95+
- leading_newlines_in_multiline_strings
96+
- library_annotations
97+
- library_prefixes
98+
- library_private_types_in_public_api
99+
- lines_longer_than_80_chars
100+
- literal_only_boolean_expressions
101+
- matching_super_parameters
102+
- missing_code_block_language_in_doc_comment
103+
- missing_whitespace_between_adjacent_strings
104+
- no_adjacent_strings_in_list
105+
- no_default_cases
106+
- no_duplicate_case_values
107+
- no_leading_underscores_for_library_prefixes
108+
- no_leading_underscores_for_local_identifiers
109+
- no_literal_bool_comparisons
110+
- no_logic_in_create_state
111+
- no_runtimeType_toString
112+
- no_self_assignments
113+
- no_wildcard_variable_uses
114+
- non_constant_identifier_names
115+
- noop_primitive_operations
116+
- null_check_on_nullable_type_parameter
117+
- null_closures
118+
- omit_local_variable_types
119+
- one_member_abstracts
120+
- only_throw_errors
121+
- overridden_fields
122+
- package_names
123+
- parameter_assignments
124+
- prefer_adjacent_string_concatenation
125+
- prefer_asserts_in_initializer_lists
126+
- prefer_asserts_with_message
127+
- prefer_collection_literals
128+
- prefer_conditional_assignment
129+
- prefer_const_constructors
130+
- prefer_const_constructors_in_immutables
131+
- prefer_const_declarations
132+
- prefer_const_literals_to_create_immutables
133+
- prefer_constructors_over_static_methods
134+
- prefer_contains
135+
- prefer_final_fields
136+
- prefer_final_in_for_each
137+
- prefer_final_locals
138+
- prefer_foreach
139+
- prefer_for_elements_to_map_fromIterable
140+
- prefer_function_declarations_over_variables
141+
- prefer_generic_function_type_aliases
142+
- prefer_if_elements_to_conditional_expressions
143+
- prefer_if_null_operators
144+
- prefer_initializing_formals
145+
- prefer_inlined_adds
146+
- prefer_int_literals
147+
- prefer_interpolation_to_compose_strings
148+
- prefer_is_empty
149+
- prefer_is_not_empty
150+
- prefer_is_not_operator
151+
- prefer_iterable_whereType
152+
- prefer_null_aware_method_calls
153+
- prefer_null_aware_operators
154+
- prefer_single_quotes
155+
- prefer_spread_collections
156+
- prefer_typing_uninitialized_variables
157+
- provide_deprecation_message
158+
- public_member_api_docs
159+
- recursive_getters
160+
- require_trailing_commas
161+
- secure_pubspec_urls
162+
- sized_box_for_whitespace
163+
- sized_box_shrink_expand
164+
- slash_for_doc_comments
165+
- sort_child_properties_last
166+
- sort_constructors_first
167+
- sort_pub_dependencies
168+
- sort_unnamed_constructors_first
169+
- specify_nonobvious_property_types
170+
- strict_top_level_inference
171+
- switch_on_type
172+
- test_types_in_equals
173+
- throw_in_finally
174+
- tighten_type_of_initializing_formals
175+
- type_annotate_public_apis
176+
- type_init_formals
177+
- type_literal_in_constant_pattern
178+
- unawaited_futures
179+
- unintended_html_in_doc_comment
180+
- unnecessary_await_in_return
181+
- unnecessary_breaks
182+
- unnecessary_brace_in_string_interps
183+
- unnecessary_const
184+
- unnecessary_constructor_name
185+
- unnecessary_getters_setters
186+
- unnecessary_ignore
187+
- unnecessary_lambdas
188+
- unnecessary_late
189+
- unnecessary_library_directive
190+
- unnecessary_library_name
191+
- unnecessary_new
192+
- unnecessary_null_aware_assignments
193+
- unnecessary_null_aware_operator_on_extension_on_nullable
194+
- unnecessary_null_checks
195+
- unnecessary_null_in_if_null_operators
196+
- unnecessary_nullable_for_final_variable_declarations
197+
- unnecessary_overrides
198+
- unnecessary_parenthesis
199+
- unnecessary_raw_strings
200+
- unnecessary_statements
201+
- unnecessary_string_escapes
202+
- unnecessary_string_interpolations
203+
- unnecessary_this
204+
- unnecessary_to_list_in_spreads
205+
- unnecessary_unawaited
206+
- unnecessary_underscores
207+
- unreachable_from_main
208+
- unrelated_type_equality_checks
209+
- use_build_context_synchronously
210+
- use_colored_box
211+
- use_enums
212+
- use_full_hex_values_for_flutter_colors
213+
- use_function_type_syntax_for_parameters
214+
- use_if_null_to_convert_nulls_to_bools
215+
- use_is_even_rather_than_modulo
216+
- use_key_in_widget_constructors
217+
- use_late_for_private_fields_and_variables
218+
- use_named_constants
219+
- use_null_aware_elements
220+
- use_raw_strings
221+
- use_rethrow_when_possible
222+
- use_setters_to_change_properties
223+
- use_string_buffers
224+
- use_string_in_part_of_directives
225+
- use_super_parameters
226+
- use_test_throws_matchers
227+
- use_to_and_as_if_applicable
228+
- use_truncating_division
229+
- valid_regexps
230+
- void_checks

lib/analysis_options.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
include: package:very_good_analysis/analysis_options.10.0.0.yaml
1+
include: package:very_good_analysis/analysis_options.10.1.0.yaml

pubspec.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,4 +8,4 @@ documentation: https://github.com/VeryGoodOpenSource/very_good_analysis
88
topics: [lints, analyzer, analysis]
99

1010
environment:
11-
sdk: ^3.9.0
11+
sdk: ^3.10.0

0 commit comments

Comments
 (0)