Skip to content

Commit e41f9f9

Browse files
[ci] Clean up analysis options (flutter#7068)
Removes some options that are no longer necessary, further aligning the options with flutter/packages. Part of flutter/flutter#113764
1 parent 3d81a00 commit e41f9f9

File tree

1 file changed

+5
-34
lines changed

1 file changed

+5
-34
lines changed

analysis_options.yaml

Lines changed: 5 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -1,50 +1,21 @@
1-
# This is a copy (as of March 2021) of flutter/flutter's analysis_options file,
2-
# with minimal changes for this repository. The goal is to move toward using a
3-
# shared set of analysis options as much as possible, and eventually a shared
4-
# file.
5-
61
# Specify analysis options.
72
#
8-
# For a list of lints, see: http://dart-lang.github.io/linter/lints/
9-
# See the configuration guide for more
10-
# https://github.com/dart-lang/sdk/tree/main/pkg/analyzer#configuring-the-analyzer
11-
#
12-
# There are other similar analysis options files in the flutter repos,
13-
# which should be kept in sync with this file:
14-
#
15-
# - analysis_options.yaml (this file)
16-
# - packages/flutter/lib/analysis_options_user.yaml
17-
# - https://github.com/flutter/flutter/blob/master/analysis_options.yaml
18-
# - https://github.com/flutter/engine/blob/main/analysis_options.yaml
19-
# - https://github.com/flutter/packages/blob/main/analysis_options.yaml
20-
#
21-
# This file contains the analysis options used for code in the flutter/plugins
22-
# repository.
3+
# This file is a copy of analysis_options.yaml from flutter repo
4+
# as of 2022-07-27, but with some modifications marked with
5+
# "DIFFERENT FROM FLUTTER/FLUTTER" below. The file is expected to
6+
# be kept in sync with the master file from the flutter repo.
237

248
analyzer:
259
language:
2610
strict-casts: true
2711
strict-raw-types: true
2812
errors:
29-
# treat missing required parameters as a warning (not a hint)
30-
missing_required_param: warning
31-
# treat missing returns as a warning (not a hint)
32-
missing_return: warning
33-
# allow having TODO comments in the code
34-
todo: ignore
3513
# allow self-reference to deprecated members (we do this because otherwise we have
3614
# to annotate every member in every test, assert, etc, when we deprecate something)
3715
deprecated_member_use_from_same_package: ignore
38-
# Ignore analyzer hints for updating pubspecs when using Future or
39-
# Stream and not importing dart:async
40-
# Please see https://github.com/flutter/flutter/pull/24528 for details.
41-
sdk_version_async_exported_from_core: ignore
4216
# Turned off until null-safe rollout is complete.
4317
unnecessary_null_comparison: ignore
44-
### Local flutter/plugins changes ###
45-
# Allow null checks for as long as mixed mode is officially supported.
46-
always_require_non_null_named_parameters: false # not needed with nnbd
47-
exclude:
18+
exclude: # DIFFERENT FROM FLUTTER/FLUTTER
4819
# Ignore generated files
4920
- '**/*.g.dart'
5021
- '**/*.mocks.dart' # Mockito @GenerateMocks

0 commit comments

Comments
 (0)