Disable diagnostic_describe_all_properties rule (#10)

solid-yuriiprykhodko · web-flow · commit c77dbc0896c0 · 2022-01-26T12:20:00.000+02:00
* Remove diagnostic_describe_all_properties rule

* Update changelog

* Add cyclomatic-complexity bug note
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,6 @@
 ## 0.0.9
 
+- Remove `diagnostic_describe_all_properties` rule
 - Disable cyclomatic complexity metric for tests
 
 ## 0.0.8
diff --git a/lib/analysis_options.yaml b/lib/analysis_options.yaml
@@ -128,7 +128,7 @@ linter:
     - curly_braces_in_flow_control_structures
     - depend_on_referenced_packages
     - deprecated_consistency
-    - diagnostic_describe_all_properties
+    # - diagnostic_describe_all_properties
     - directives_ordering
     - do_not_use_environment
     - empty_catches
diff --git a/lib/analysis_options_test.yaml b/lib/analysis_options_test.yaml
@@ -14,6 +14,7 @@ dart_code_metrics:
 # Since we're not using the source-lines-of-code rule, `main()` function in test can 
 # have high cyclomatic complexity.
 # For rationale against splitting up `main()` in tests, see `source-lines-of-code` comments.
+# Also there is a bug in metric calculation: https://github.com/dart-code-checker/dart-code-metrics/issues/663
     cyclomatic-complexity: false
 
   rules:
