Skip to content

Commit

Permalink
Fix Docker container names in pre-commit hooks
Browse files Browse the repository at this point in the history
The previous Docker container names contained Bash syntax, but
pre-commit interprets them as string literals, resulting in invalid
Docker container names. Remove Bash syntax, and replace it with string
literals.
  • Loading branch information
mwarres committed Oct 6, 2022
1 parent 2a6ddcd commit d7fb453
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .pre-commit-hooks.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
require_serial: true
args:
- mega-linter-runner
- --container-name "megalinter-$(basename "$PWD")"
- --containername megalinter-incremental
- --remove-container
- --fix
- --env
Expand Down Expand Up @@ -35,7 +35,7 @@
require_serial: true
args:
- mega-linter-runner
- --container-name "megalinter-all-$(basename "$PWD")"
- --containername megalinter-full
- --remove-container
- --fix
- --env
Expand Down
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,9 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),

Note: Can be used with `oxsecurity/megalinter@beta` in your GitHub Action mega-linter.yml file, or with `oxsecurity/megalinter:beta` docker image

- Fix invalid Docker container names in .pre-commit-hooks.yaml ([#1932](https://github.com/oxsecurity/megalinter/issues/1932))
- Correct removeContainer casing in runner ([#1917](https://github.com/oxsecurity/megalinter/issues/1917))

- Linter versions upgrades
- [cfn-lint](https://github.com/aws-cloudformation/cfn-lint) from 0.66.0 to **0.66.1** on 2022-10-04
- [mypy](https://mypy.readthedocs.io/en/stable/) from 0.981 to **0.982** on 2022-10-04
Expand Down

0 comments on commit d7fb453

Please sign in to comment.