Skip to content

Commit

Permalink
exclude ansible lints checks
Browse files Browse the repository at this point in the history
  • Loading branch information
unkcpz committed Dec 4, 2023
1 parent 01d0bac commit 92991c4
Show file tree
Hide file tree
Showing 4 changed files with 28 additions and 20 deletions.
1 change: 0 additions & 1 deletion .ansible-lint
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,3 @@ warn_list: # or 'skip_list' to silence them completely
- '301' # Commands should not change things if nothing needs doing
- '305' # Use shell only when shell functionality is required
- '504' # Do not use 'local_action', use 'delegate_to: localhost'
- syntax-check
18 changes: 9 additions & 9 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,16 +9,16 @@ on:

jobs:

#pre-commit:
# runs-on: ubuntu-latest
pre-commit:
runs-on: ubuntu-latest

# steps:
# - uses: actions/checkout@v3
# - name: Set up Python
# uses: actions/setup-python@v4
# with:
# python-version: 3.8
# - uses: pre-commit/[email protected]
steps:
- uses: actions/checkout@v3
- name: Set up Python
uses: actions/setup-python@v4
with:
python-version: 3.10
- uses: pre-commit/[email protected]

# docs:
# runs-on: ubuntu-latest
Expand Down
2 changes: 2 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -22,3 +22,5 @@ submodules/roles/marvel-nccr*
submodules/roles/gantsign*
submodules/roles/geerlingguy*
submodules/ansible_collections/

local/modules/__pycache__/*.pyc
27 changes: 17 additions & 10 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,24 +2,31 @@
# See usage instructions at https://pre-commit.com
repos:

- repo: https://github.com/macisamuele/language-formatters-pre-commit-hooks
rev: v2.3.0
hooks:
- id: pretty-format-yaml
args: [--autofix, --indent, "2", --preserve-quotes]

- repo: https://github.com/adrienverge/yamllint
rev: v1.26.3
rev: v1.33.0
hooks:
- id: yamllint

- repo: https://github.com/ansible/ansible-lint
rev: v6.3.0
rev: v6.22.1
hooks:
- id: ansible-lint
args: [--exclude, roles/]
# these should bring back after fixing the ansible-lint issues
# we exclude most of the roles and tasks, as they are not update to the new ansible version
args: [
--exclude,
roles/,
submodules/roles/,
playbook-aiidalab-qe.yml,
playbook-build.yml,
playbook-package.yml,
.pre-commit-config.yaml,
local/tasks/,
.github/workflows/ci.yml,
submodules/ansible_collections/,
]

- repo: https://github.com/psf/black
rev: 23.1.0
rev: 23.11.0
hooks:
- id: black

0 comments on commit 92991c4

Please sign in to comment.