diff --git a/CHANGELOG.md b/CHANGELOG.md index 85f9ac6c87..2bd4a7a614 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -4,6 +4,7 @@ * fix: [`avoid-border-all`](https://dartcodemetrics.dev/docs/rules/flutter/avoid-border-all) is triggered even when it is not a const. * fix: remove duplicated and ignore void function calls for [`prefer-moving-to-variable`](https://dartcodemetrics.dev/docs/rules/common/prefer-moving-to-variable). +* fix: temporary remove enums support for [`prefer-trailing-comma`](https://dartcodemetrics.dev/docs/rules/common/prefer-trailing-comma). ## 4.15.1 diff --git a/lib/src/analyzers/lint_analyzer/rules/rules_list/prefer_trailing_comma/visitor.dart b/lib/src/analyzers/lint_analyzer/rules/rules_list/prefer_trailing_comma/visitor.dart index 40a5d91141..6404c01fe7 100644 --- a/lib/src/analyzers/lint_analyzer/rules/rules_list/prefer_trailing_comma/visitor.dart +++ b/lib/src/analyzers/lint_analyzer/rules/rules_list/prefer_trailing_comma/visitor.dart @@ -28,13 +28,6 @@ class _Visitor extends RecursiveAstVisitor { ); } - @override - void visitEnumDeclaration(EnumDeclaration node) { - super.visitEnumDeclaration(node); - - _visitNodeList(node.constants, node.leftBracket, node.rightBracket); - } - @override void visitListLiteral(ListLiteral node) { super.visitListLiteral(node); diff --git a/test/src/analyzers/lint_analyzer/rules/rules_list/prefer_trailing_comma/examples/incorrect_example.dart b/test/src/analyzers/lint_analyzer/rules/rules_list/prefer_trailing_comma/examples/incorrect_example.dart index 25b2d53b94..dd3c5b1150 100644 --- a/test/src/analyzers/lint_analyzer/rules/rules_list/prefer_trailing_comma/examples/incorrect_example.dart +++ b/test/src/analyzers/lint_analyzer/rules/rules_list/prefer_trailing_comma/examples/incorrect_example.dart @@ -35,7 +35,7 @@ enum FirstEnum { thirdItem, forthItem, fifthItem, - sixthItem // LINT + sixthItem } class FirstClass { diff --git a/test/src/analyzers/lint_analyzer/rules/rules_list/prefer_trailing_comma/prefer_trailing_comma_rule_test.dart b/test/src/analyzers/lint_analyzer/rules/rules_list/prefer_trailing_comma/prefer_trailing_comma_rule_test.dart index 650ee7335a..a609837dcb 100644 --- a/test/src/analyzers/lint_analyzer/rules/rules_list/prefer_trailing_comma/prefer_trailing_comma_rule_test.dart +++ b/test/src/analyzers/lint_analyzer/rules/rules_list/prefer_trailing_comma/prefer_trailing_comma_rule_test.dart @@ -30,8 +30,8 @@ void main() { RuleTestHelper.verifyIssues( issues: issues, - startLines: [3, 9, 13, 18, 24, 28, 38, 49, 58, 64, 70], - startColumns: [50, 7, 5, 52, 9, 8, 3, 59, 3, 3, 3], + startLines: [3, 9, 13, 18, 24, 28, 49, 58, 64, 70], + startColumns: [50, 7, 5, 52, 9, 8, 59, 3, 3, 3], locationTexts: [ 'String thirdArgument', "'and another string for length exceed'", @@ -39,7 +39,6 @@ void main() { 'String thirdArgument', "'and another string for length exceed'", "'some other string'", - 'sixthItem', 'this.forthField', "'and another string for length exceed'", "'and another string for length exceed'", @@ -56,7 +55,6 @@ void main() { 'Prefer trailing comma.', 'Prefer trailing comma.', 'Prefer trailing comma.', - 'Prefer trailing comma.', ], replacementComments: [ 'Add trailing comma.', @@ -69,7 +67,6 @@ void main() { 'Add trailing comma.', 'Add trailing comma.', 'Add trailing comma.', - 'Add trailing comma.', ], replacements: [ 'String thirdArgument,', @@ -78,7 +75,6 @@ void main() { 'String thirdArgument,', "'and another string for length exceed',", "'some other string',", - 'sixthItem,', 'this.forthField,', "'and another string for length exceed',", "'and another string for length exceed',", @@ -102,8 +98,8 @@ void main() { RuleTestHelper.verifyIssues( issues: issues, - startLines: [9, 17, 19, 37, 41, 75, 91, 99, 109, 119], - startColumns: [21, 33, 20, 23, 19, 18, 43, 21, 19, 19], + startLines: [9, 17, 19, 37, 41, 91, 99, 109, 119], + startColumns: [21, 33, 20, 23, 19, 43, 21, 19, 19], locationTexts: [ 'String arg1', 'void Function() callback', @@ -114,7 +110,6 @@ void main() { '() {\n' ' return;\n' ' }', - 'firstItem', '0', '\'some string\'', '\'some string\'', @@ -130,7 +125,6 @@ void main() { 'Prefer trailing comma.', 'Prefer trailing comma.', 'Prefer trailing comma.', - 'Prefer trailing comma.', ], ); });