From c4744b6a1cd517d7e21ef2754272091282c16db3 Mon Sep 17 00:00:00 2001 From: Joshua Magady Date: Sat, 4 Mar 2023 17:00:51 -0600 Subject: [PATCH] fix(pre-commit): corrected pre-commit args for megalinter-full (#2411) * fix(pre-commit): corrected pre-commit args containernam argument contained bot the flag and value on the same line. These have to be separate line * chore(changelog): add entry to changelog * fix(changelog): corrected spelling * chore(linter): add drbothen to word list --- .github/linters/.cspell.json | 3 ++- .pre-commit-hooks.yaml | 3 ++- CHANGELOG.md | 1 + 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/.github/linters/.cspell.json b/.github/linters/.cspell.json index a117e6c1c64..cffeefeee38 100644 --- a/.github/linters/.cspell.json +++ b/.github/linters/.cspell.json @@ -93,6 +93,7 @@ "Crefrange", "Cres", "Csrf", + "drbothen", "C\u00e9dric", "DARTANALYZER", "DEVSKIM", @@ -1374,4 +1375,4 @@ "\u0636clojure", "\u077bcbad" ] -} \ No newline at end of file +} diff --git a/.pre-commit-hooks.yaml b/.pre-commit-hooks.yaml index 9ede209ee40..2b240eac60a 100644 --- a/.pre-commit-hooks.yaml +++ b/.pre-commit-hooks.yaml @@ -36,7 +36,8 @@ require_serial: true args: - mega-linter-runner - - --containername megalinter-full + - --containername + - "megalinter-full" - --remove-container - --fix - --env diff --git a/CHANGELOG.md b/CHANGELOG.md index 999052b9b20..d19a17104d5 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -27,6 +27,7 @@ Note: Can be used with `oxsecurity/megalinter@beta` in your GitHub Action mega-l - Added **cli_lint_fix_arg_name** parameter to **powershell formatter** descriptor as without it, autofix does not work, by @bdovaz in [#2294](https://github.com/oxsecurity/megalinter/pull/2294) - Created **ProtolintLinter** class to fix the problem that returns exit code 1 when it encounters a problem to correct even though it corrects it correctly, by @bdovaz in [#2294](https://github.com/oxsecurity/megalinter/pull/2294) - Concatenate **--output** parameter correctly to **xmllint** linter, by @bdovaz in [#2294](https://github.com/oxsecurity/megalinter/pull/2294) + - Modified the .pre-commit-hooks.yaml for megalinter-full so the containername argument is correctly split between two lines, by @drbothen [#2411](https://github.com/oxsecurity/megalinter/pull/2411) - Documentation - Change **swiftlint** example that did not correctly reflect the **--fix** parameter, by @bdovaz in [#2294](https://github.com/oxsecurity/megalinter/pull/2294)