Skip to content

Commit ce3302e

Browse files
gnpricechrisbobbe
authored andcommitted
lint: Re-enable analyzer on Pigeon-generated code
The upstream issue that made this exclusion necessary has been fixed: flutter/flutter#145633 In fact it turns out it was fixed in pigeon 18.0.0, which we took in 8df3064 back in April; we just didn't notice at the time. Meanwhile this file grew a different analyzer issue during the time it wasn't getting checked -- and that one was an actual actionable one for us to fix, where a doc comment still referred to a deprecated bit of API. Fixed that in the previous commit. Glad to have the analyzer re-enabled here.
1 parent 402f998 commit ce3302e

File tree

1 file changed

+0
-9
lines changed

1 file changed

+0
-9
lines changed

analysis_options.yaml

-9
Original file line numberDiff line numberDiff line change
@@ -3,15 +3,6 @@
33
include: package:flutter_lints/flutter.yaml
44

55
analyzer:
6-
exclude:
7-
# Skip analysis on Pigeon-generated code, because it currently
8-
# triggers some stylistic lints and they aren't actionable for us.
9-
# (Some lints could signal a bug, which would be good to catch...
10-
# but typically those are lints Flutter upstream keeps enabled, so
11-
# if Pigeon tripped them it'd immediately get caught.)
12-
# TODO(pigeon) re-enable lints once clean: https://github.com/flutter/flutter/issues/145633
13-
- lib/host/*.g.dart
14-
156
language:
167
strict-inference: true
178
strict-raw-types: true

0 commit comments

Comments
 (0)