Skip to content

Commit

Permalink
Merge branch 'update-tooling'
Browse files Browse the repository at this point in the history
  • Loading branch information
PeterJCLaw committed Sep 16, 2023
2 parents d7e553f + 8a738fb commit d5f2d28
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 11 deletions.
4 changes: 0 additions & 4 deletions .ansible-lint
Original file line number Diff line number Diff line change
@@ -1,8 +1,4 @@
skip_list:
- 305
- 401
- 301
- 503
- fqcn-builtins
- meta-no-info
- role-name
Expand Down
9 changes: 3 additions & 6 deletions dev-requirements.txt
Original file line number Diff line number Diff line change
@@ -1,6 +1,3 @@
ansible==6.4.0
ansible-lint==6.8.0
yamllint==1.28.0

# ansible-lint 6.8 isn't compatible with ansible-compat>=4
ansible-compat==3.0.2
ansible==8.4.0
ansible-lint==6.19.0
yamllint==1.32.0
2 changes: 1 addition & 1 deletion roles/code-submitter/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@
notify:
Reload nginx

- name: Install database
- name: Install database # noqa: no-changed-when - We want to always run this (it handles its own idempotency)
shell: # noqa: command-instead-of-shell - We need this to use `environment`
argv:
- "{{ venv_dir }}/bin/alembic"
Expand Down

0 comments on commit d5f2d28

Please sign in to comment.