Skip to content

Commit

Permalink
Merge pull request #33 from cisagov/improvement/update_pre-commit_hooks
Browse files Browse the repository at this point in the history
Update pre-commit hooks (and flip Actions cache order in workflow).
  • Loading branch information
mcdonnnj authored Feb 12, 2020
2 parents 8116a89 + d99fd00 commit 16872bf
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 13 deletions.
12 changes: 6 additions & 6 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,12 +14,6 @@ jobs:
- uses: actions/setup-python@v1
with:
python-version: 3.8
- name: Cache pre-commit hooks
uses: actions/cache@v1
with:
path: ~/.cache/pre-commit
key: "${{ runner.os }}-pre-commit-\
${{ hashFiles('**/.pre-commit-config.yaml') }}"
- name: Cache pip test requirements
uses: actions/cache@v1
with:
Expand All @@ -29,6 +23,12 @@ jobs:
restore-keys: |
${{ runner.os }}-pip-test-
${{ runner.os }}-pip-
- name: Cache pre-commit hooks
uses: actions/cache@v1
with:
path: ~/.cache/pre-commit
key: "${{ runner.os }}-pre-commit-\
${{ hashFiles('**/.pre-commit-config.yaml') }}"
- name: Install dependencies
run: |
python -m pip install --upgrade pip
Expand Down
14 changes: 7 additions & 7 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ default_language_version:

repos:
- repo: https://github.com/pre-commit/pre-commit-hooks
rev: v2.4.0
rev: v2.5.0
hooks:
- id: check-executables-have-shebangs
- id: check-json
Expand All @@ -27,13 +27,13 @@ repos:
- id: requirements-txt-fixer
- id: trailing-whitespace
- repo: https://github.com/igorshubovych/markdownlint-cli
rev: v0.19.0
rev: v0.22.0
hooks:
- id: markdownlint
args:
- --config=.mdl_config.json
- repo: https://github.com/adrienverge/yamllint
rev: v1.18.0
rev: v1.20.0
hooks:
- id: yamllint
- repo: https://github.com/detailyang/pre-commit-shell
Expand All @@ -47,7 +47,7 @@ repos:
additional_dependencies:
- flake8-docstrings
- repo: https://github.com/asottile/pyupgrade
rev: v1.25.1
rev: v1.26.2
hooks:
- id: pyupgrade
- repo: https://github.com/PyCQA/bandit
Expand All @@ -61,7 +61,7 @@ repos:
hooks:
- id: black
- repo: https://github.com/asottile/seed-isort-config
rev: v1.9.3
rev: v1.9.4
hooks:
- id: seed-isort-config
- repo: https://github.com/pre-commit/mirrors-isort
Expand All @@ -71,7 +71,7 @@ repos:
hooks:
- id: isort
- repo: https://github.com/ansible/ansible-lint.git
rev: v4.1.1a5
rev: v4.2.0
hooks:
- id: ansible-lint
# files: molecule/default/playbook.yml
Expand All @@ -81,7 +81,7 @@ repos:
- id: terraform_fmt
- id: terraform_validate_no_variables
- repo: https://github.com/IamTheFij/docker-pre-commit
rev: v1.0.0
rev: v1.0.1
hooks:
- id: docker-compose-check
- repo: https://github.com/prettier/prettier
Expand Down

0 comments on commit 16872bf

Please sign in to comment.