feat: change rules, metrics and anti-patterns doc urls for website urls (#414)

* feat: change rules, metrics and anti-patterns doc urls for website urls

* fix: change rule type for new rule

Author: incendial

lib/src/analyzers/lint_analyzer/anti_patterns/pattern_utils.dart

@@ -24,15 +24,11 @@ Issue createIssue({
 /// Returns a url of a page containing documentation associated with [pattern]
 Uri documentation(Pattern pattern) => Uri(
       scheme: 'https',
-      host: 'github.com',
+      host: 'dartcodemetrics.dev',
       pathSegments: [
-        'dart-code-checker',
-        'dart-code-metrics',
-        'blob',
-        'master',
-        'doc',
+        'docs',
         'anti-patterns',
-        '${pattern.id}.md',
+        pattern.id,
       ],
     );
 

lib/src/analyzers/lint_analyzer/metrics/metric_utils.dart

@@ -3,15 +3,11 @@ import 'models/metric_value_level.dart';
 /// Returns the url of a page containing documentation associated with [metricId]
 Uri documentation(String metricId) => Uri(
       scheme: 'https',
-      host: 'github.com',
+      host: 'dartcodemetrics.dev',
       pathSegments: [
-        'dart-code-checker',
-        'dart-code-metrics',
-        'blob',
-        'master',
-        'doc',
+        'docs',
         'metrics',
-        '$metricId.md',
+        metricId,
       ],
     );
 

lib/src/analyzers/lint_analyzer/rules/models/angular_rule.dart

@@ -0,0 +1,19 @@
+import '../../models/severity.dart';
+import 'rule.dart';
+import 'rule_documentation.dart';
+import 'rule_type.dart';
+
+abstract class AngularRule extends Rule {
+  const AngularRule({
+    required String id,
+    required RuleDocumentation documentation,
+    required Severity severity,
+    required Iterable<String> excludes,
+  }) : super(
+          id: id,
+          type: RuleType.angular,
+          documentation: documentation,
+          severity: severity,
+          excludes: excludes,
+        );
+}

lib/src/analyzers/lint_analyzer/rules/models/common_rule.dart

@@ -0,0 +1,19 @@
+import '../../models/severity.dart';
+import 'rule.dart';
+import 'rule_documentation.dart';
+import 'rule_type.dart';
+
+abstract class CommonRule extends Rule {
+  const CommonRule({
+    required String id,
+    required RuleDocumentation documentation,
+    required Severity severity,
+    required Iterable<String> excludes,
+  }) : super(
+          id: id,
+          type: RuleType.common,
+          documentation: documentation,
+          severity: severity,
+          excludes: excludes,
+        );
+}

lib/src/analyzers/lint_analyzer/rules/models/flutter_rule.dart

@@ -0,0 +1,19 @@
+import '../../models/severity.dart';
+import 'rule.dart';
+import 'rule_documentation.dart';
+import 'rule_type.dart';
+
+abstract class FlutterRule extends Rule {
+  const FlutterRule({
+    required String id,
+    required RuleDocumentation documentation,
+    required Severity severity,
+    required Iterable<String> excludes,
+  }) : super(
+          id: id,
+          type: RuleType.flutter,
+          documentation: documentation,
+          severity: severity,
+          excludes: excludes,
+        );
+}

lib/src/analyzers/lint_analyzer/rules/models/intl_rule.dart

@@ -0,0 +1,19 @@
+import '../../models/severity.dart';
+import 'rule.dart';
+import 'rule_documentation.dart';
+import 'rule_type.dart';
+
+abstract class IntlRule extends Rule {
+  const IntlRule({
+    required String id,
+    required RuleDocumentation documentation,
+    required Severity severity,
+    required Iterable<String> excludes,
+  }) : super(
+          id: id,
+          type: RuleType.intl,
+          documentation: documentation,
+          severity: severity,
+          excludes: excludes,
+        );
+}

lib/src/analyzers/lint_analyzer/rules/models/rule.dart

@@ -2,6 +2,7 @@ import '../../models/internal_resolved_unit_result.dart';
 import '../../models/issue.dart';
 import '../../models/severity.dart';
 import 'rule_documentation.dart';
+import 'rule_type.dart';
 
 /// An interface to communicate with a rules
 ///
@@ -10,6 +11,9 @@ abstract class Rule {
   /// The id of the rule.
   final String id;
 
+  /// The type of the rule.
+  final RuleType type;
+
   /// The documentation associated with the rule
   final RuleDocumentation documentation;
 
@@ -22,6 +26,7 @@ abstract class Rule {
   /// Initialize a newly created [Rule].
   const Rule({
     required this.id,
+    required this.type,
     required this.documentation,
     required this.severity,
     required this.excludes,

lib/src/analyzers/lint_analyzer/rules/models/rule_type.dart

@@ -0,0 +1,11 @@
+/// Represents a rule type. Used by documentation path construction.
+class RuleType {
+  final String value;
+
+  const RuleType._(this.value);
+
+  static const common = RuleType._('common');
+  static const flutter = RuleType._('flutter');
+  static const intl = RuleType._('intl');
+  static const angular = RuleType._('angular');
+}

lib/src/analyzers/lint_analyzer/rules/rule_utils.dart

@@ -28,15 +28,12 @@ Issue createIssue({
 /// Returns a url of a page containing documentation associated with [rule]
 Uri documentation(Rule rule) => Uri(
       scheme: 'https',
-      host: 'github.com',
+      host: 'dartcodemetrics.dev',
       pathSegments: [
-        'dart-code-checker',
-        'dart-code-metrics',
-        'blob',
-        'master',
-        'doc',
+        'docs',
         'rules',
-        '${rule.id}.md',
+        rule.type.value,
+        rule.id,
       ],
     );
 

lib/src/analyzers/lint_analyzer/rules/rules_list/always_remove_listener/always_remove_listener.dart

@@ -9,13 +9,13 @@ import '../../../models/internal_resolved_unit_result.dart';
 import '../../../models/issue.dart';
 import '../../../models/severity.dart';
 import '../../flutter_rule_utils.dart';
-import '../../models/rule.dart';
+import '../../models/flutter_rule.dart';
 import '../../models/rule_documentation.dart';
 import '../../rule_utils.dart';
 
 part 'visitor.dart';
 
-class AlwaysRemoveListenerRule extends Rule {
+class AlwaysRemoveListenerRule extends FlutterRule {
   static const String ruleId = 'always-remove-listener';
 
   static const _warningMessage =