Skip to content

Commit

Permalink
ansible-lint.conf: remove no-tabs exclusion
Browse files Browse the repository at this point in the history
The rule "no-tabs" was skipped because a playbook needed to be refactor.
This is done now and the rule should not be skipped anymore.

Signed-off-by: Erwann Roussy <[email protected]>
  • Loading branch information
eroussy committed May 30, 2023
1 parent e375e39 commit 7e151da
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions ansible-lint.conf
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@ skip_list:
- unnamed-task # All tasks should be named
- role-name # All role names should match "^[a-z_][a-z0-9_]*$"
- risky-file-permissions # All file creation must specify permissions
- no-tabs # Most files should not contain tabs
- no-handler # "when: result.changed" should trigger a handler instead
- no-changed-when # Commands should not change things if nothing needs doing
- no-relative-paths # Doesn't need a relative path in role
Expand All @@ -36,9 +35,6 @@ skip_list:
# permissions on files must be done on the overall SEAPATH project in order to
# be effective. This is a much bigger task.
#
# - no-tabs is raised by a patch task. The concerned tool should soon merge the
# patch and this will not be required anymore.
#
# - no-handler, no-changed-when and no-relative-paths should not be skipped. The
# raised warnings should be corrected as soon as possible for these three rules to
# run on the CI.

0 comments on commit 7e151da

Please sign in to comment.