Enforce override annotations in dart2js

Update the excludes in the test directory analysis options to match the
files that were not updated to add override annotations.
Update the excludes in `testing.json` to match the analysis options.

With a consistent analysis options test config the diagnostics in
editors will match what is enforced by the test.

Also clean up the no longer required `enableSuperMixins` config.

Change-Id: If0e23642c25e0b5c1aced6a5a8c8c05a6da538f8
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/96836
Reviewed-by: Sigmund Cherem <sigmund@google.com>
Commit-Queue: Nate Bosch <nbosch@google.com>
This commit is contained in:
Nate Bosch 2019-03-14 18:47:05 +00:00 committed by commit-bot@chromium.org
parent 5e1397fc4e
commit 28d697a6c1
3 changed files with 14 additions and 6 deletions

View file

@ -10,3 +10,7 @@ analyzer:
deprecated_member_use: ignore
# Allow deprecated calls from within the same package
deprecated_member_use_from_same_package: ignore
linter:
rules:
- annotate_overrides

View file

@ -14,7 +14,9 @@
],
"exclude": [
"^tests/compiler/dart2js/inference/data/super_invoke\\.dart"
"^tests/compiler/dart2js/.*/data/.*",
"^tests/compiler/dart2js/.*/model_data/.*",
"^tests/compiler/dart2js/deferred_loading/libs/.*"
]
}
}

View file

@ -3,13 +3,15 @@
# BSD-style license that can be found in the LICENSE file.
analyzer:
language:
enableSuperMixins: false
errors:
todo: ignore
deprecated_member_use: ignore
exclude:
- data/*
- sourcemaps/data/*
- '**/data/*'
- '**/model_data/*'
- 'deferred_loading/libs/*'
linter:
rules:
- annotate_overrides