Skip to content

Commit

Permalink
fix(pre-commit): corrected pre-commit args for megalinter-full (#2411)
Browse files Browse the repository at this point in the history
* 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
  • Loading branch information
drbothen authored Mar 4, 2023
1 parent 2e5e577 commit c4744b6
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
3 changes: 2 additions & 1 deletion .github/linters/.cspell.json
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,7 @@
"Crefrange",
"Cres",
"Csrf",
"drbothen",
"C\u00e9dric",
"DARTANALYZER",
"DEVSKIM",
Expand Down Expand Up @@ -1374,4 +1375,4 @@
"\u0636clojure",
"\u077bcbad"
]
}
}
3 changes: 2 additions & 1 deletion .pre-commit-hooks.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,8 @@
require_serial: true
args:
- mega-linter-runner
- --containername megalinter-full
- --containername
- "megalinter-full"
- --remove-container
- --fix
- --env
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down

0 comments on commit c4744b6

Please sign in to comment.