fix: ignore override methods for avoid-redundant-async (#1060)

Author: incendial

CHANGELOG.md

@@ -14,6 +14,7 @@
 * feat: **Breaking change** cleanup public API.
 * chore: restrict `analyzer` version to `>=5.1.0 <5.3.0`.
 * feat: add `print-config` option to all commands.
+* fix: ignore `@override` methods for [`avoid-redundant-async`](https://dartcodemetrics.dev/docs/rules/common/avoid-redundant-async).
 
 ## 4.21.2
 

lib/src/analyzers/lint_analyzer/rules/rules_list/avoid_redundant_async/visitor.dart

@@ -9,7 +9,12 @@ class _Visitor extends RecursiveAstVisitor<void> {
   void visitMethodDeclaration(MethodDeclaration node) {
     super.visitMethodDeclaration(node);
 
-    if (_hasRedundantAsync(node.body)) {
+    final isOverride = node.metadata.any(
+      (node) =>
+          node.name.name == 'override' && node.atSign.type == TokenType.AT,
+    );
+
+    if (!isOverride && _hasRedundantAsync(node.body)) {
       _declarations.add(node);
     }
   }

test/src/analyzers/lint_analyzer/rules/rules_list/avoid_redundant_async/examples/example.dart

@@ -21,6 +21,9 @@ class SomeClass {
   // LINT
   Future<String> anotherAsyncMethod() async => Future.value('value');
 
+  @override
+  Future<String> overrideMethod() async => Future.value('value');
+
   Future<String> someAsyncMethod(Future<String> later) => later;
 
   Future<void> report(Iterable<String> records) async {