Merge branch 'release-v0.1.0' into proper_super_calls

README.md

@@ -31,15 +31,8 @@ Then you can see suggestions in your IDE or you can run checks manually:
 
 ```bash
 dart analyze;
-dart run dart_code_metrics:metrics analyze lib test;
-dart run dart_code_metrics:metrics check-unused-files lib test;
-dart run dart_code_metrics:metrics check-unused-l10n lib test;
-
 ```
-Beware that some of the `dart_code_metrics` checks are not displayed in IDE so running checks 
-manually or in your actions (CI) is essential.
 
-Learn more: https://github.com/dart-code-checker/dart-code-metrics#cli
 # Badge
 
 To indicate that your project is using Solid Lints, you can use the following badge:

lib/lints/no_magic_number/no_magic_number_rule.dart

@@ -69,7 +69,9 @@ class NoMagicNumberRule extends SolidLintRule<NoMagicNumberParameters> {
           .where(_isNotInsideConstConstructor)
           .where(_isNotInDateTime)
           .where(_isNotInsideIndexExpression)
-          .where(_isNotInsideEnumConstantArguments);
+          .where(_isNotInsideEnumConstantArguments)
+          .where(_isNotDefaultValue)
+          .where(_isNotInConstructorInitializer);
 
       for (final magicNumber in magicNumbers) {
         reporter.reportErrorForNode(code, magicNumber);
@@ -122,4 +124,12 @@ class NoMagicNumberRule extends SolidLintRule<NoMagicNumberParameters> {
       null;
 
   bool _isNotInsideIndexExpression(Literal l) => l.parent is! IndexExpression;
+
+  bool _isNotDefaultValue(Literal literal) {
+    return literal.thisOrAncestorOfType<DefaultFormalParameter>() == null;
+  }
+
+  bool _isNotInConstructorInitializer(Literal literal) {
+    return literal.thisOrAncestorOfType<ConstructorInitializer>() == null;
+  }
 }

lint_test/no_magic_number_test.dart

@@ -1,4 +1,7 @@
-// ignore_for_file: unused_local_variable, prefer_match_file_name
+// ignore_for_file: unused_local_variable
+// ignore_for_file: prefer_match_file_name
+// ignore_for_file: avoid_unused_parameters
+// ignore_for_file: no_empty_block
 
 /// Check the `no_magic_number` rule
 
@@ -52,6 +55,34 @@ void fun() {
   // Allowed in indexed expression
   final result = list[1];
 
-  // Allowed in DateTime because it doesn't have cons constructor
+  // Allowed in DateTime because it doesn't have const constructor
   final apocalypse = DateTime(2012, 12, 21);
 }
+
+// Allowed for defaults in constructors and methods.
+class DefaultValues {
+  final int value;
+
+  DefaultValues.named({
+    this.value = 2,
+  });
+
+  DefaultValues.positional([
+    this.value = 3,
+  ]);
+
+  void methodWithNamedParam({int value = 4}) {}
+
+  void methodWithPositionalParam([int value = 5]) {}
+}
+
+void topLevelFunctionWithDefaultParam({int value = 6}) {
+  ({int value = 7}) {};
+}
+
+// Allowed for numbers in constructor initializer.
+class ConstructorInitializer {
+  final int value;
+
+  ConstructorInitializer() : value = 10;
+}