refactor: Enable new DCM rule: avoid-cascade-after-if-null (#2676)

Enable new DCM rule: avoid-cascade-after-if-null Reference: https://dcm.dev/docs/rules/common/avoid-cascade-after-if-null/
flame-engine · Aug 26, 2023 · 158fc34 · 158fc34
1 parent 10e4109
commit 158fc34
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/packages/flame_jenny/jenny/test/test_scenario.dart b/packages/flame_jenny/jenny/test/test_scenario.dart
@@ -19,8 +19,7 @@ Future<void> testScenario({
  List<String>? commands,
  YarnProject? yarn,
 }) async {
- final yarnProject = yarn ?? YarnProject()
- ..strictCharacterNames = false;
+ final yarnProject = (yarn ?? YarnProject())..strictCharacterNames = false;
  commands?.forEach(yarnProject.commands.addOrphanedCommand);
 
  Future<void> testBody() async {

diff --git a/packages/flame_lint/lib/analysis_options_with_dcm.yaml b/packages/flame_lint/lib/analysis_options_with_dcm.yaml
@@ -6,5 +6,6 @@ dart_code_metrics:
  rules:
  # dart rules
  - avoid-banned-imports
+ - avoid-cascade-after-if-null
  # flutter rules
  - prefer-define-hero-tag