feat(eslint-plugin-template)!: add prefer-control-flow to recommended config

JamesHenry · JamesHenry · commit 77cd24d3fdb0 · 2025-11-22T17:32:22.000+04:00
diff --git a/packages/angular-eslint/src/configs/template-recommended.ts b/packages/angular-eslint/src/configs/template-recommended.ts
@@ -19,6 +19,7 @@ export default (
       '@angular-eslint/template/banana-in-box': 'error',
       '@angular-eslint/template/eqeqeq': 'error',
       '@angular-eslint/template/no-negated-async': 'error',
+      '@angular-eslint/template/prefer-control-flow': 'error',
     },
   },
 ];
diff --git a/packages/eslint-plugin-template/README.md b/packages/eslint-plugin-template/README.md
@@ -67,7 +67,7 @@ Please see https://github.com/angular-eslint/angular-eslint for full usage instr
 | [`prefer-at-empty`](https://github.com/angular-eslint/angular-eslint/blob/main/packages/eslint-plugin-template/docs/rules/prefer-at-empty.md) | Prefer using `@empty` with `@for` loops instead of a separate `@if` or `@else` block to reduce code and make it easier to read. |  | :wrench: |  |  |
 | [`prefer-built-in-pipes`](https://github.com/angular-eslint/angular-eslint/blob/main/packages/eslint-plugin-template/docs/rules/prefer-built-in-pipes.md) | Encourages the use of Angular built-in pipes (e.g. lowercase, uppercase, titlecase) instead of certain JavaScript methods in Angular templates. |  |  |  |  |
 | [`prefer-contextual-for-variables`](https://github.com/angular-eslint/angular-eslint/blob/main/packages/eslint-plugin-template/docs/rules/prefer-contextual-for-variables.md) | Ensures that contextual variables are used in @for blocks where possible instead of aliasing them. |  | :wrench: |  |  |
-| [`prefer-control-flow`](https://github.com/angular-eslint/angular-eslint/blob/main/packages/eslint-plugin-template/docs/rules/prefer-control-flow.md) | Ensures that the built-in control flow is used. |  |  |  |  |
+| [`prefer-control-flow`](https://github.com/angular-eslint/angular-eslint/blob/main/packages/eslint-plugin-template/docs/rules/prefer-control-flow.md) | Ensures that the built-in control flow is used. | :white_check_mark: |  |  |  |
 | [`prefer-ngsrc`](https://github.com/angular-eslint/angular-eslint/blob/main/packages/eslint-plugin-template/docs/rules/prefer-ngsrc.md) | Ensures ngSrc is used instead of src for img elements |  |  | :bulb: |  |
 | [`prefer-template-literal`](https://github.com/angular-eslint/angular-eslint/blob/main/packages/eslint-plugin-template/docs/rules/prefer-template-literal.md) | Ensure that template literals are used instead of concatenating strings or expressions. |  | :wrench: |  |  |
 | [`role-has-required-aria`](https://github.com/angular-eslint/angular-eslint/blob/main/packages/eslint-plugin-template/docs/rules/role-has-required-aria.md) | [Accessibility] Ensures elements with ARIA roles have all required properties for that role. |  |  | :bulb: | :accessibility: |
diff --git a/packages/eslint-plugin-template/src/configs/recommended.json b/packages/eslint-plugin-template/src/configs/recommended.json
@@ -4,6 +4,7 @@
   "rules": {
     "@angular-eslint/template/banana-in-box": "error",
     "@angular-eslint/template/eqeqeq": "error",
-    "@angular-eslint/template/no-negated-async": "error"
+    "@angular-eslint/template/no-negated-async": "error",
+    "@angular-eslint/template/prefer-control-flow": "error"
   }
 }
diff --git a/packages/eslint-plugin-template/src/rules/prefer-control-flow.ts b/packages/eslint-plugin-template/src/rules/prefer-control-flow.ts
@@ -12,6 +12,7 @@ export default createESLintRule<Options, MessageIds>({
     type: 'suggestion',
     docs: {
       description: 'Ensures that the built-in control flow is used.',
+      recommended: 'recommended',
     },
     schema: [],
     messages: {

Original file line number	Diff line number	Diff line change
`@@ -4,6 +4,7 @@`
`4`	`4`	`"rules": {`
`5`	`5`	`"@angular-eslint/template/banana-in-box": "error",`
`6`	`6`	`"@angular-eslint/template/eqeqeq": "error",`
`7`		`- "@angular-eslint/template/no-negated-async": "error"`
	`7`	`+ "@angular-eslint/template/no-negated-async": "error",`
	`8`	`+ "@angular-eslint/template/prefer-control-flow": "error"`
`8`	`9`	`}`
`9`	`10`	`}`