Skip to content

Conversation

@Jkhall81
Copy link
Contributor

@Jkhall81 Jkhall81 commented Dec 31, 2025

Fixes a context-blindness issue where the no-tabs rule incorrectly flagged valid tab characters in lineinfile tasks when they were nested inside a block. By refactoring matchtask to validate keys gainst the allow_list globally within the task structure, the rule now correctly identifies whitelisted parameters (like regexp or line) regardless of nesting depth.

Fixes #4719

@Jkhall81 Jkhall81 requested a review from a team as a code owner December 31, 2025 17:15
@Jkhall81 Jkhall81 force-pushed the fix/4719-no-tabs-block-nesting branch from bbd6c5f to b415e9a Compare December 31, 2025 17:19
@Jkhall81 Jkhall81 force-pushed the fix/4719-no-tabs-block-nesting branch from b415e9a to 69a154b Compare December 31, 2025 17:21
@Jkhall81 Jkhall81 changed the title Fix/4719 no tabs block nesting Fix: allow valid tabs in lineinfile nested in blocks (#4719) Dec 31, 2025
@Jkhall81 Jkhall81 force-pushed the fix/4719-no-tabs-block-nesting branch from 69a154b to 736a7f9 Compare December 31, 2025 17:27
@Jkhall81 Jkhall81 changed the title Fix: allow valid tabs in lineinfile nested in blocks (#4719) fix: allow valid tabs in lineinfile nested in blocks (#4719) Dec 31, 2025
@Jkhall81 Jkhall81 force-pushed the fix/4719-no-tabs-block-nesting branch from 736a7f9 to c7b832c Compare December 31, 2025 17:36
@github-actions github-actions bot added the fix label Dec 31, 2025
@Jkhall81 Jkhall81 force-pushed the fix/4719-no-tabs-block-nesting branch from c7b832c to 4c42db3 Compare December 31, 2025 17:56
@github-actions github-actions bot added fix and removed fix labels Dec 31, 2025
@Jkhall81 Jkhall81 force-pushed the fix/4719-no-tabs-block-nesting branch from 4c42db3 to 2ad6390 Compare December 31, 2025 18:20
@github-actions github-actions bot added fix and removed fix labels Dec 31, 2025
@Jkhall81 Jkhall81 force-pushed the fix/4719-no-tabs-block-nesting branch from 2ad6390 to 7d4e3e0 Compare December 31, 2025 18:31
@github-actions github-actions bot added fix and removed fix labels Dec 31, 2025
@github-actions github-actions bot added fix and removed fix labels Jan 5, 2026
@ssbarnea ssbarnea merged commit f6f8393 into ansible:main Jan 6, 2026
21 checks passed
@github-project-automation github-project-automation bot moved this from Review to Done in 🧰 devtools project board Jan 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

no-tabs Rule Does Not Recognize lineinfile Inside block

3 participants