Skip to content

Commit 14d2edb

Browse files
authored
Merge pull request #93 from cisagov/improvement/update_ansible-lint_pre-commit_version
Update the `ansible-lint` Version in the pre-commit Configuration
2 parents f3000e5 + a3c5aa7 commit 14d2edb

File tree

2 files changed

+23
-3
lines changed

2 files changed

+23
-3
lines changed

.ansible-lint

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
---
2+
# See https://ansible-lint.readthedocs.io/en/latest/configuring.html
3+
# for a list of the configuration elements that can exist in this
4+
# file.
5+
enable_list:
6+
# Useful checks that one must opt-into. See here for more details:
7+
# https://ansible-lint.readthedocs.io/en/latest/rules.html
8+
- fcqn-builtins
9+
- no-log-password
10+
- no-same-owner
11+
exclude_paths:
12+
# This exclusion is implicit, unless exclude_paths is defined
13+
- .cache
14+
# Seems wise to ignore this too
15+
- .github
16+
# ansible-lint doesn't like the role name in this playbook, but it's
17+
# what molecule requires
18+
- molecule/default/converge.yml
19+
# These two are Molecule configuration files, not Ansible playbooks
20+
- molecule/default/molecule-no-systemd.yml
21+
- molecule/default/molecule-with-systemd.yml
22+
use_default_rules: true

.pre-commit-config.yaml

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -105,9 +105,7 @@ repos:
105105

106106
# Ansible hooks
107107
- repo: https://github.com/ansible-community/ansible-lint
108-
# This is intentionally being held back because of issues in v5 per
109-
# https://github.com/cisagov/skeleton-ansible-role/issues/69
110-
rev: v4.3.7
108+
rev: v5.2.1
111109
hooks:
112110
- id: ansible-lint
113111
# files: molecule/default/playbook.yml

0 commit comments

Comments
 (0)