Merge pull request #42 from netglade/feat/adhere-dcm-17

Author: tenhobi

.github/workflows/netglade_flutter_utils-test.yaml

@@ -25,7 +25,7 @@ jobs:
 
     steps:
       - uses: actions/checkout@v4
-      - uses: subosito/flutter-action@v2.12.0
+      - uses: subosito/flutter-action@v2.16.0
 
       - name: Install Dependencies
         run: flutter pub get

packages/netglade_analysis/CHANGELOG.md

@@ -1,3 +1,15 @@
+## 11.0.0
+- Adhere to DCM 1.17.0:
+  - Add avoid-collection-equality-checks
+  - Add avoid-multi-assignment
+  - Add prefer-switch-with-sealed-classes
+  - Add prefer-single-declaration-per-file
+  - Add prefer-use-callback
+  - Add avoid-misused-hooks
+  - Add avoid-ref-read-inside-build
+  - Add avoid-ref-watch-outside-build
+  - Add avoid-unnecessary-consumer-widgets
+
 ## 10.0.0
 - Adhere to DCM 1.16.0:
   - Add avoid-conditions-with-boolean-literals

packages/netglade_analysis/README.md

@@ -23,7 +23,7 @@ To use the lints, add as a dev dependency in your `pubspec.yaml`:
 
 ```yaml
 dev_dependencies:
-  netglade_analysis: ^9.0.0
+  netglade_analysis: ^11.0.0
 ```
 
 Then, add an include in `analysis_options.yaml`:

packages/netglade_analysis/lib/dcm.yaml

@@ -1,4 +1,4 @@
-# version 1.16.0 - https://dcm.dev/changelog
+# version 1.17.0 - https://dcm.dev/changelog
 dart_code_metrics:
   rules:
     ### Common
@@ -16,6 +16,7 @@ dart_code_metrics:
     - avoid-cascade-after-if-null
     - avoid-casting-to-extension-type
     - avoid-collapsible-if
+    - avoid-collection-equality-checks
     - avoid-collection-methods-with-unrelated-types
     # - avoid-collection-mutating-methods # sometimes it's ok
     - avoid-conditions-with-boolean-literals
@@ -70,6 +71,7 @@ dart_code_metrics:
     - avoid-missing-enum-constant-in-map
     - avoid-misused-wildcard-pattern
     - avoid-mixing-named-and-positional-fields
+    - avoid-multi-assignment
     # - avoid-mutating-parameters # we use ValueNotifier etc.
     # - avoid-negated-conditions # sometimes it might be better
     - avoid-nested-conditional-expressions:
@@ -147,6 +149,7 @@ dart_code_metrics:
     - avoid-unused-parameters
     - avoid-weak-cryptographic-algorithms
     - avoid-wildcard-cases-with-enums
+    - prefer-switch-with-sealed-classes
     # - banned-usage # * OK, we just don't have global config
     - binary-expression-operand-order
     - double-literal-format
@@ -272,6 +275,8 @@ dart_code_metrics:
     - prefer-return-await
     # - prefer-returning-conditional-expressions # annoying sometimes
     - prefer-simpler-patterns-null-check
+    - prefer-single-declaration-per-file:
+        ignore-private: true
     - prefer-specific-cases-first
     - prefer-static-class:
         ignore-private: true
@@ -354,6 +359,11 @@ dart_code_metrics:
     - prefer-correct-bloc-provider
     - prefer-multi-bloc-provider
 
+    ### Riverpod
+    - avoid-ref-read-inside-build
+    - avoid-ref-watch-outside-build
+    - avoid-unnecessary-consumer-widgets
+
     ### Equatable
     # - extend-equatable # * OK, we just don't have global config
     - list-all-equatable-fields
@@ -378,7 +388,9 @@ dart_code_metrics:
     ### Flutter Hooks
     - avoid-conditional-hooks
     - avoid-hooks-outside-build
+    - avoid-misused-hooks
     - avoid-unnecessary-hook-widgets
+    - prefer-use-callback
     - prefer-use-prefix
 
     ### Intl - not used

packages/netglade_analysis/pubspec.yaml

@@ -1,5 +1,5 @@
 name: netglade_analysis
-version: 10.0.0
+version: 11.0.0
 description: Lint rules for Dart and Flutter used internally at netglade.
 repository: https://github.com/netglade/flutter_core/tree/main/packages/netglade_analysis
 issue_tracker: https://github.com/netglade/flutter_core/issues