Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(109)

Unified Diff: pkg/analyzer_cli/lib/src/options.dart

Issue 2874043002: Add --fatal-infos; hide --fatal-hints. (Closed)
Patch Set: Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « pkg/analyzer_cli/lib/src/driver.dart ('k') | pkg/analyzer_cli/test/options_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/analyzer_cli/lib/src/options.dart
diff --git a/pkg/analyzer_cli/lib/src/options.dart b/pkg/analyzer_cli/lib/src/options.dart
index 90db6efe822039cedd6cd88f061f42673b6db802..9d754768a635105e035dc112430c391e024adcbc 100644
--- a/pkg/analyzer_cli/lib/src/options.dart
+++ b/pkg/analyzer_cli/lib/src/options.dart
@@ -98,9 +98,6 @@ class CommandLineOptions {
/// errors.
final bool enableTypeChecks;
- /// Whether to treat hints as fatal
- final bool hintsAreFatal;
-
/// Whether to ignore unrecognized flags
final bool ignoreUnrecognizedFlags;
@@ -135,6 +132,9 @@ class CommandLineOptions {
/// Whether to treat warnings as fatal
final bool warningsAreFatal;
+ /// Whether to treat info level items as fatal
+ final bool infosAreFatal;
+
/// Whether to use strong static checking.
final bool strongMode;
@@ -144,8 +144,7 @@ class CommandLineOptions {
/// Whether implicit dynamic is enabled (mainly for strong mode users)
final bool implicitDynamic;
- // TODO(devoncarew): Do we need this flag? Shouldn't we go by the severity of
- // the lint?
+ // TODO(devoncarew): Deprecate and remove this flag.
/// Whether to treat lints as fatal
final bool lintsAreFatal;
@@ -180,7 +179,6 @@ class CommandLineOptions {
displayVersion = args['version'],
enableTypeChecks = args['enable_type_checks'],
enableAssertInitializer = args['enable-assert-initializers'],
- hintsAreFatal = args['fatal-hints'],
ignoreUnrecognizedFlags = args['ignore-unrecognized-flags'],
lints = args[lintsFlag],
log = args['log'],
@@ -193,11 +191,12 @@ class CommandLineOptions {
showPackageWarningsPrefix = args['x-package-warnings-prefix'],
showSdkWarnings = args['sdk-warnings'],
_sourceFiles = args.rest,
+ infosAreFatal = args['fatal-infos'] || args['fatal-hints'],
warningsAreFatal = args['fatal-warnings'],
+ lintsAreFatal = args['fatal-lints'],
strongMode = args['strong'],
implicitCasts = !args['no-implicit-casts'],
implicitDynamic = !args['no-implicit-dynamic'],
- lintsAreFatal = args['fatal-lints'],
useAnalysisDriverMemoryByteStore =
args['use-analysis-driver-memory-byte-store'],
verbose = args['verbose'],
@@ -341,18 +340,12 @@ class CommandLineOptions {
help: 'Do not show hint results.',
defaultsTo: false,
negatable: false)
- ..addFlag('fatal-hints',
- help: 'Treat hints as fatal.', defaultsTo: false, negatable: false)
+ ..addFlag('fatal-infos',
+ help: 'Treat infos as fatal.', defaultsTo: false, negatable: false)
..addFlag('fatal-warnings',
help: 'Treat non-type warnings as fatal.',
defaultsTo: false,
negatable: false)
- ..addFlag('fatal-lints',
- help: 'Treat lints as fatal.', defaultsTo: false, negatable: false)
- ..addFlag('package-warnings',
- help: 'Show warnings from package: imports.',
- defaultsTo: false,
- negatable: false)
..addFlag('help',
abbr: 'h',
help:
@@ -489,6 +482,21 @@ class CommandLineOptions {
defaultsTo: false,
negatable: false,
hide: hide)
+ ..addFlag('fatal-hints',
+ help: 'Treat hints as fatal (deprecated: use --fatal-infos).',
+ defaultsTo: false,
+ negatable: false,
+ hide: hide)
+ ..addFlag('fatal-lints',
+ help: 'Treat lints as fatal.',
+ defaultsTo: false,
+ negatable: false,
+ hide: hide)
+ ..addFlag('package-warnings',
+ help: 'Show warnings from package: imports.',
+ defaultsTo: false,
+ negatable: false,
+ hide: hide)
..addOption('url-mapping',
help: '--url-mapping=libraryUri,/path/to/library.dart directs the '
'analyzer to use "library.dart" as the source for an import '
« no previous file with comments | « pkg/analyzer_cli/lib/src/driver.dart ('k') | pkg/analyzer_cli/test/options_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698