-
-
Notifications
You must be signed in to change notification settings - Fork 244
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MegaLinter v6 has been released ! #1592
Comments
Félicitations! Et merci de perpétuer l'esprit OSS malgré le changement de giron de Megalinter 👍 |
The repo path change broke all my github actions... Congrats on V6 and the new managers! |
What did break exactly ? |
Redirects don't work for actions at all, by design: As per docs: |
@SyncDK when I migrated from nvuillam to megalinter it worked like a charm, I did not expect this one, sorry 😭 |
Yeah, it appears that GitHub changed this behavior between the time of the two migrations. I didn't notice since we use the pre-commit hook. |
Running
Doing |
@andrewvaughan I think you have a previous version in cache. Please can you try And if it still does not work, you can try |
That worked, thanks! |
The repo path changes broke our GitHub actions as well, but we made slight change in the workflow. Followed this template |
@mohsin996 that's what It also adds |
This issue has been automatically marked as stale because it has not had recent activity. If you think this issue should stay open, please remove the |
@nvuillam Is there any reason why Using locally to test my migration from v5 to v6
FYI: was disabled in context of PR #1153 (see https://github.com/oxsecurity/megalinter/blame/main/megalinter/descriptors/markdown.megalinter-descriptor.yml#L49) |
@nvuillam would it be possible to reconsider adding |
@onemec rstfmt is back in beta version, and soon in next release :) |
This issue has been automatically marked as stale because it has not had recent activity. If you think this issue should stay open, please remove the |
This issue has been automatically marked as stale because it has not had recent activity. If you think this issue should stay open, please remove the |
This issue has been automatically marked as stale because it has not had recent activity. If you think this issue should stay open, please remove the |
After being promised for months now, MegaLinter v6 has finally been released :)
You can see all the updates in the release notes -> https://github.com/oxsecurity/megalinter/releases/tag/v6.0.0
Just run
npx mega-linter-runner@latest --upgrade
to use it :) (troubleshooting)Some linters has been removed, so if you referred to them in your configuration, you'll have to remove them
DOCKERFILE_DOCKERFILELINT
: It was not maintained anymore and hadolint much more efficientRST_RSTFMT
: It was highly experimental and not maintained anymoreOn some big repos, some of the new linters can be slow (like REPOSITORY_GITLEAKS) or too noisy ( like REPOSITORY_SEMGREP), don't hesitate to disable them using
DISABLE_LINTERS
property in.mega-linter.yml
: You only can decide the linters you need !If you see any bug/issue, please let us know !
Best regards
MegaLinter maintainers & OX Security team
Wanna know why MegaLinter has joined OX Security ? Please read the following article :)
https://nicolas.vuillamy.fr/megalinter-sells-his-soul-and-joins-ox-security-2a91a0027628
Edit: Known issues:
Redirection of github action megalinter/megalinter to oxsecurity/megalinter
megalinter/megalinter
byoxsecurity/megalinter
, even if you don't want to migrate to v6 yetPRE_COMMANDS using
npm install
. Fixed in v6.0.4cd /node-deps && npm install xxx ...
. This will be automatically added soon (post-css module installation #1258)mega-linter-runner does not upgrade report folder for uploading artifacts (mega-linter-runner --upgrade does not update report folder in Github Action Workflows #1609) Fixed in v6.0.5
The text was updated successfully, but these errors were encountered: