Skip to content
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

Implement fallback in case git diff does not work with merge-base #3503

Merged
merged 2 commits into from
Apr 23, 2024

Conversation

nvuillam
Copy link
Member

Fixes #3501
Fixes #3421

Copy link
Contributor

github-actions bot commented Apr 23, 2024

🦙 MegaLinter status: ⚠️ WARNING

Descriptor Linter Files Fixed Errors Elapsed time
✅ BASH bash-exec 5 0 0.01s
✅ BASH shellcheck 5 0 0.13s
✅ BASH shfmt 5 0 0 0.18s
✅ COPYPASTE jscpd yes no 3.88s
✅ DOCKERFILE hadolint 128 0 15.14s
✅ JSON jsonlint 19 0 0.23s
✅ JSON v8r 21 0 26.93s
✅ MAKEFILE checkmake 1 0 0.01s
⚠️ MARKDOWN markdownlint 262 0 266 31.29s
✅ MARKDOWN markdown-table-formatter 262 0 0 135.12s
✅ OPENAPI spectral 1 0 1.63s
⚠️ PYTHON bandit 209 64 3.08s
✅ PYTHON black 209 0 0 5.06s
✅ PYTHON flake8 209 0 1.52s
✅ PYTHON isort 209 0 0 1.25s
✅ PYTHON mypy 209 0 20.24s
✅ PYTHON pylint 209 0 17.54s
✅ PYTHON ruff 209 0 0 0.23s
✅ REPOSITORY checkov yes no 40.25s
✅ REPOSITORY git_diff yes no 0.19s
⚠️ REPOSITORY grype yes 1 24.0s
✅ REPOSITORY secretlint yes no 17.49s
✅ REPOSITORY trivy yes no 29.92s
✅ REPOSITORY trivy-sbom yes no 11.68s
⚠️ REPOSITORY trufflehog yes 1 8.91s
✅ SPELL cspell 686 0 29.31s
⚠️ SPELL lychee 342 1 24.45s
✅ XML xmllint 3 0 0 0.17s
✅ YAML prettier 160 0 0 5.84s
✅ YAML v8r 102 0 186.02s
✅ YAML yamllint 161 0 2.09s

See detailed report in MegaLinter reports

MegaLinter is graciously provided by OX Security

@nvuillam nvuillam merged commit eac233f into main Apr 23, 2024
130 of 131 checks passed
@nvuillam nvuillam deleted the fixes/git-diff-fallback branch April 23, 2024 18:18
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.

[7.11] possible regression : need support —env VALIDATE_ALL_CODEBASE=false does not work in docker
1 participant