Skip to content

Commit c4802f3

Browse files
refactor: change error severity from none to ignore
1 parent 0c74ca3 commit c4802f3

File tree

5 files changed

+8
-9
lines changed

5 files changed

+8
-9
lines changed

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -238,7 +238,7 @@ custom_lint:
238238
my_lint_rule: error
239239
```
240240

241-
The available severity levels are: `error`, `warning`, `info`, and `none`.
241+
The available severity levels are: `error`, `warning`, `info`, and `ignore`.
242242

243243
### Obtaining the list of lints in the CI
244244

packages/custom_lint/test/analysis_options_test.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ Analyzing...
121121
expect(process.exitCode, 1);
122122
});
123123

124-
test('correctly applies none severity from analysis_options.yaml',
124+
test('correctly applies ignore severity from analysis_options.yaml',
125125
() async {
126126
final plugin = createTestPlugin();
127127

@@ -130,7 +130,7 @@ Analyzing...
130130
analysisOptions: '''
131131
custom_lint:
132132
errors:
133-
test_lint: none
133+
test_lint: ignore
134134
''',
135135
);
136136

packages/custom_lint_builder/lib/src/custom_analyzer_converter.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ class CustomAnalyzerConverter {
8484
// Config severities override processor severities
8585
final severity = configSeverity ?? processor?.severity;
8686

87-
// Errors with none severity are filtered out.
87+
// Errors with ignore severity are filtered out.
8888
if (severity == analyzer.ErrorSeverity.NONE) {
8989
continue;
9090
}

packages/custom_lint_core/lib/src/configs.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,7 @@ class CustomLintConfigs {
118118
'info' => ErrorSeverity.INFO,
119119
'warning' => ErrorSeverity.WARNING,
120120
'error' => ErrorSeverity.ERROR,
121-
'none' => ErrorSeverity.NONE,
121+
'ignore' => ErrorSeverity.NONE,
122122
_ => throw UnsupportedError(
123123
'Unsupported severity ${entry.value} for key: ${entry.key}',
124124
),

packages/custom_lint_core/test/configs_test.dart

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -366,7 +366,7 @@ custom_lint:
366366
rule_name_1: error
367367
rule_name_2: warning
368368
rule_name_3: info
369-
rule_name_4: none
369+
rule_name_4: ignore
370370
''');
371371
final configs = CustomLintConfigs.parse(analysisOptions, packageConfig);
372372

@@ -387,11 +387,10 @@ custom_lint:
387387
expect(
388388
() => CustomLintConfigs.parse(analysisOptions, packageConfig),
389389
throwsA(
390-
isArgumentError.having(
390+
isA<UnsupportedError>().having(
391391
(e) => e.message,
392392
'message',
393-
'Provided error severity: invalid_severity specified for key: rule_name_1 is not valid. '
394-
'Valid error severities are: none, info, warning, error',
393+
'Unsupported severity invalid_severity for key: rule_name_1',
395394
),
396395
),
397396
);

0 commit comments

Comments
 (0)