Skip to content

Commit

Permalink
Don't add literal " character to filenames (#1942)
Browse files Browse the repository at this point in the history
The extra quotes caused MegaLinter to fail to find the first and last
files in the list, because the first has a literal double quote
character prepended to it, and the last has one appended to it.
  • Loading branch information
Kurt-von-Laven committed Oct 20, 2022
1 parent 6048ba6 commit 32bb60f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ 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

- Don't add literal double quote character to filenames in mega-linter-runner ([#1942](https://github.com/oxsecurity/megalinter/issues/1942)).
- Remove default npm-groovy-lint extra arguments ([#1872](https://github.com/oxsecurity/megalinter/issues/1872))

- Linter versions upgrades
Expand Down
2 changes: 1 addition & 1 deletion mega-linter-runner/lib/runner.js
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,7 @@ ERROR: Docker engine has not been found on your system.
if ((options._ || []).length > 0) {
commandArgs.push(
...["-e"],
`MEGALINTER_FILES_TO_LINT="${options._.join(",")}"`
`MEGALINTER_FILES_TO_LINT=${options._.join(",")}`
);
}
commandArgs.push(dockerImage);
Expand Down

0 comments on commit 32bb60f

Please sign in to comment.