-
Notifications
You must be signed in to change notification settings - Fork 146
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
Log Review #431
Log Review #431
Conversation
@KnifMelti is that ok with your work? 😇 |
🦙 MegaLinter status:
|
Descriptor | Linter | Files | Fixed | Errors | Elapsed time |
---|---|---|---|---|---|
jscpd | yes | 9 | 2.4s | ||
✅ POWERSHELL | powershell | 6 | 0 | 17.2s | |
✅ POWERSHELL | powershell_formatter | 6 | 0 | 10.68s | |
checkov | yes | 1 | 10.77s | ||
devskim | yes | 4 | 2.78s | ||
✅ REPOSITORY | dustilock | yes | no | 0.02s | |
✅ REPOSITORY | gitleaks | yes | no | 0.61s | |
git_diff | yes | 1 | 0.01s | ||
✅ REPOSITORY | grype | yes | no | 12.57s | |
✅ REPOSITORY | kics | yes | no | 14.08s | |
✅ REPOSITORY | secretlint | yes | no | 1.28s | |
✅ REPOSITORY | syft | yes | no | 0.27s | |
✅ REPOSITORY | trivy | yes | no | 3.77s | |
✅ REPOSITORY | trivy-sbom | yes | no | 1.29s | |
trufflehog | yes | 1 | 3.95s | ||
cspell | 7 | 73 | 9.18s |
See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true
in mega-linter.yml to validate all sources, not only the diff
You said tomorrow 😍 |
yes, but I want it in the nightly built of tonight ^^ |
seams OK so far on my side. I merge, will see tomorrow for troubleshoot in case something is broken |
if everything is good, I'll work on ServiceUI. I have a plan to onboard it with Github Actions. This way, it comes from the MDT .msi and it only picks serviceUI.exe that is very light.
|
What an intricate world we live in nowadays... 🤪 |
Proposed Changes