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

[automation] Auto-update linters version, help and documentation #3204

Merged
merged 2 commits into from
Dec 11, 2023

Conversation

nvuillam
Copy link
Member

[automation] Auto-update linters version, help and documentation

Copy link
Contributor

github-actions bot commented Dec 10, 2023

🦙 MegaLinter status: ⚠️ WARNING

Descriptor Linter Files Fixed Errors Elapsed time
✅ BASH bash-exec 5 0 0.09s
✅ BASH shellcheck 5 0 0.21s
✅ BASH shfmt 5 0 0 0.51s
✅ COPYPASTE jscpd yes no 4.54s
✅ DOCKERFILE hadolint 127 0 11.92s
✅ JSON eslint-plugin-jsonc 23 0 0 4.12s
✅ JSON jsonlint 21 0 0.26s
✅ JSON v8r 23 0 18.98s
✅ MAKEFILE checkmake 1 0 0.0s
⚠️ MARKDOWN markdownlint 259 0 265 29.62s
✅ MARKDOWN markdown-link-check 259 0 8.7s
✅ MARKDOWN markdown-table-formatter 259 0 0 65.57s
✅ OPENAPI spectral 1 0 1.81s
⚠️ PYTHON bandit 206 59 3.69s
✅ PYTHON black 206 0 0 5.04s
✅ PYTHON flake8 206 0 1.62s
✅ PYTHON isort 206 0 0 1.17s
✅ PYTHON mypy 206 0 15.02s
✅ PYTHON pylint 206 0 18.59s
⚠️ PYTHON pyright 206 346 26.19s
✅ PYTHON ruff 206 0 0 0.6s
✅ REPOSITORY checkov yes no 37.04s
✅ REPOSITORY git_diff yes no 0.53s
⚠️ REPOSITORY grype yes 1 24.51s
✅ REPOSITORY secretlint yes no 12.26s
✅ REPOSITORY trivy yes no 31.46s
✅ REPOSITORY trivy-sbom yes no 25.94s
⚠️ REPOSITORY trufflehog yes 1 16.47s
✅ SPELL cspell 679 0 31.49s
✅ SPELL lychee 339 0 4.96s
✅ XML xmllint 3 0 0 0.5s
✅ YAML prettier 160 0 0 6.87s
✅ YAML v8r 102 0 158.62s
✅ YAML yamllint 161 0 2.17s

See detailed report in MegaLinter reports

MegaLinter is graciously provided by OX Security

@@ -25,7 +25,7 @@ To apply file formatting you must set `XML_XMLLINT_CLI_LINT_MODE: file` and `XML
| Variable | Description | Default value |
|-----------------------------------------|---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|--------------------|
| XML_XMLLINT_AUTOFORMAT | If set to `true`, it will reformat and reindent the output | `false` |
| XML_XMLLINT_INDENT | The number of indentation spaces when `XML_XMLLINT_AUTOFORMAT` is `true` | ` ` |
| XML_XMLLINT_INDENT | The number of indentation spaces when `XML_XMLLINT_AUTOFORMAT` is `true` | `` |
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This auto formatting seems problematic, but is not present after a force-push, maybe merged by another PR as is. It does not mean the same thing.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there other places where this happens?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's the build doc & autofix that fight each other, happens for months , maybe years, it's not nice but not a big deal ^^

What is more wrong is that I forced the merge of a tiny PR and I didn't think that latest version of checkov would be broken :/

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Posted a bug... :/

bridgecrewio/checkov#5841

@nvuillam nvuillam merged commit 6f1f578 into main Dec 11, 2023
130 checks passed
@nvuillam nvuillam deleted the create-pull-request/patch branch December 11, 2023 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants