Skip to content

Fix linter disabled reason usage#4466

Merged
nvuillam merged 3 commits intomainfrom
fix-disabled-reason-usage
Jan 9, 2025
Merged

Fix linter disabled reason usage#4466
nvuillam merged 3 commits intomainfrom
fix-disabled-reason-usage

Conversation

@bdovaz
Copy link
Copy Markdown
Collaborator

@bdovaz bdovaz commented Jan 8, 2025

No description provided.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jan 8, 2025

🦙 MegaLinter status: ⚠️ WARNING

Descriptor Linter Files Fixed Errors Elapsed time
✅ API spectral 1 0 1.88s
⚠️ BASH bash-exec 6 1 0.07s
✅ BASH shellcheck 6 0 0.31s
✅ BASH shfmt 6 0 0 0.91s
✅ COPYPASTE jscpd yes no 3.71s
✅ DOCKERFILE hadolint 129 0 26.25s
✅ JSON jsonlint 20 0 0.24s
✅ JSON v8r 22 0 14.46s
⚠️ MARKDOWN markdownlint 267 0 301 27.84s
✅ MARKDOWN markdown-table-formatter 267 0 0 161.85s
⚠️ PYTHON bandit 214 66 3.57s
✅ PYTHON black 214 0 0 4.38s
✅ PYTHON flake8 214 0 2.15s
✅ PYTHON isort 214 0 0 1.14s
✅ PYTHON mypy 214 0 18.98s
✅ PYTHON pylint 214 0 36.31s
✅ PYTHON ruff 214 0 0 0.55s
✅ REPOSITORY checkov yes no 39.2s
✅ REPOSITORY git_diff yes no 0.82s
⚠️ REPOSITORY grype yes 26 13.31s
✅ REPOSITORY secretlint yes no 12.0s
✅ REPOSITORY trivy yes no 16.56s
✅ REPOSITORY trivy-sbom yes no 0.27s
⚠️ REPOSITORY trufflehog yes 1 53.56s
✅ SPELL cspell 717 0 14.75s
⚠️ SPELL lychee 349 13 7.18s
✅ XML xmllint 3 0 0 0.77s
✅ YAML prettier 160 0 0 5.46s
✅ YAML v8r 102 0 32.05s
✅ YAML yamllint 161 0 3.07s

See detailed report in MegaLinter reports

MegaLinter is graciously provided by OX Security

@bdovaz
Copy link
Copy Markdown
Collaborator Author

bdovaz commented Jan 8, 2025

@nvuillam there is some trivy error not specifically related to my changes

@nvuillam
Copy link
Copy Markdown
Member

nvuillam commented Jan 9, 2025

@bdovaz working on it :)
I'll update your branch once main will be fixed :)

@nvuillam nvuillam merged commit 3f8cf1d into main Jan 9, 2025
@nvuillam nvuillam deleted the fix-disabled-reason-usage branch January 9, 2025 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants