-
Notifications
You must be signed in to change notification settings - Fork 134
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
Review log creation / rotation #683
Conversation
🦙 MegaLinter status:
|
Descriptor | Linter | Files | Fixed | Errors | Elapsed time |
---|---|---|---|---|---|
jscpd | yes | 6 | 2.24s | ||
✅ JSON | jsonlint | 1 | 0 | 0.17s | |
✅ JSON | prettier | 1 | 0 | 0 | 0.39s |
✅ JSON | v8r | 1 | 0 | 2.68s | |
✅ POWERSHELL | powershell | 4 | 0 | 10.1s | |
✅ POWERSHELL | powershell_formatter | 4 | 0 | 10.95s | |
checkov | yes | 1 | 14.75s | ||
devskim | yes | 11 | 1.77s | ||
✅ REPOSITORY | dustilock | yes | no | 0.01s | |
✅ REPOSITORY | gitleaks | yes | no | 1.04s | |
✅ REPOSITORY | git_diff | yes | no | 0.01s | |
✅ REPOSITORY | grype | yes | no | 15.28s | |
✅ REPOSITORY | kics | yes | no | 1.57s | |
✅ REPOSITORY | secretlint | yes | no | 0.76s | |
✅ REPOSITORY | syft | yes | no | 0.41s | |
✅ REPOSITORY | trivy | yes | no | 7.66s | |
✅ REPOSITORY | trivy-sbom | yes | no | 0.83s | |
trufflehog | yes | 1 | 3.45s | ||
cspell | 6 | 11 | 4.06s | ||
✅ SPELL | lychee | 1 | 0 | 0.05s |
See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true
in mega-linter.yml to validate all sources, not only the diff
Return $False | ||
} | ||
# create new file | ||
Write-ToLog "New log file created" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@KnifMelti just to be sure I didn't break anything? It seems better to me, but if you could confirm...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That remains to be seen 😄 but you're an expert, and I'm sleeping.
Anyway, you've merged it
[MegaLinter] Apply linters automatic fixes
Proposed Changes