oxsecurity / megalinter

🦙 MegaLinter analyzes 50 languages, 22 formats, 21 tooling formats, excessive copy-pastes, spelling mistakes and security issues in your repository sources with a GitHub Action, other CI tools or locally.
https://megalinter.io
GNU Affero General Public License v3.0
1.9k stars 231 forks source link

chore(deps): update ghcr.io/terraform-linters/tflint docker tag to v0.52.0 #3712

Closed renovate[bot] closed 2 months ago

renovate[bot] commented 3 months ago

Mend Renovate

This PR contains the following updates:

Package Type Update Change
ghcr.io/terraform-linters/tflint minor 0.51.1 -> 0.52.0
ghcr.io/terraform-linters/tflint stage minor v0.51.1 -> v0.52.0

Release Notes

terraform-linters/tflint (ghcr.io/terraform-linters/tflint) ### [`v0.52.0`](https://togithub.com/terraform-linters/tflint/compare/v0.51.2...v0.52.0) [Compare Source](https://togithub.com/terraform-linters/tflint/compare/v0.51.2...v0.52.0) ### [`v0.51.2`](https://togithub.com/terraform-linters/tflint/releases/tag/v0.51.2) [Compare Source](https://togithub.com/terraform-linters/tflint/compare/v0.51.1...v0.51.2) #### What's Changed - build(deps): Bump github.com/hashicorp/go-plugin from 1.6.0 to 1.6.1 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/terraform-linters/tflint/pull/2047](https://togithub.com/terraform-linters/tflint/pull/2047) - build(deps): Bump github.com/fatih/color from 1.16.0 to 1.17.0 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/terraform-linters/tflint/pull/2048](https://togithub.com/terraform-linters/tflint/pull/2048) - build(deps): Bump golangci/golangci-lint-action from 5.3.0 to 6.0.1 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/terraform-linters/tflint/pull/2049](https://togithub.com/terraform-linters/tflint/pull/2049) - build(deps): Bump google.golang.org/grpc from 1.63.2 to 1.64.0 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/terraform-linters/tflint/pull/2053](https://togithub.com/terraform-linters/tflint/pull/2053) - build(deps): Bump github.com/hashicorp/go-version from 1.6.0 to 1.7.0 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/terraform-linters/tflint/pull/2054](https://togithub.com/terraform-linters/tflint/pull/2054) - build(deps): Bump alpine from 3.19 to 3.20 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/terraform-linters/tflint/pull/2055](https://togithub.com/terraform-linters/tflint/pull/2055) - build(deps): Bump goreleaser/goreleaser-action from 5 to 6 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/terraform-linters/tflint/pull/2061](https://togithub.com/terraform-linters/tflint/pull/2061) - build(deps): Bump golang.org/x/crypto from 0.23.0 to 0.24.0 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/terraform-linters/tflint/pull/2062](https://togithub.com/terraform-linters/tflint/pull/2062) - build(deps): Bump golang.org/x/text from 0.15.0 to 0.16.0 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/terraform-linters/tflint/pull/2064](https://togithub.com/terraform-linters/tflint/pull/2064) - build(deps): Bump golang.org/x/oauth2 from 0.20.0 to 0.21.0 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/terraform-linters/tflint/pull/2063](https://togithub.com/terraform-linters/tflint/pull/2063) - build(deps): Bump golang.org/x/net from 0.25.0 to 0.26.0 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/terraform-linters/tflint/pull/2065](https://togithub.com/terraform-linters/tflint/pull/2065) - build(deps): Bump github.com/jessevdk/go-flags from 1.5.0 to 1.6.1 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/terraform-linters/tflint/pull/2066](https://togithub.com/terraform-linters/tflint/pull/2066) - build(deps): Bump docker/build-push-action from 5 to 6 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/terraform-linters/tflint/pull/2067](https://togithub.com/terraform-linters/tflint/pull/2067) - build(deps): Bump github.com/hashicorp/hcl/v2 from 2.20.1 to 2.21.0 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/terraform-linters/tflint/pull/2068](https://togithub.com/terraform-linters/tflint/pull/2068) - build(deps): Bump github.com/go-test/deep from 1.1.0 to 1.1.1 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/terraform-linters/tflint/pull/2069](https://togithub.com/terraform-linters/tflint/pull/2069) - build(deps): Bump github.com/hashicorp/go-getter from 1.7.4 to 1.7.5 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/terraform-linters/tflint/pull/2070](https://togithub.com/terraform-linters/tflint/pull/2070) - deps: Go 1.22.4 by [@​wata727](https://togithub.com/wata727) in [https://github.com/terraform-linters/tflint/pull/2073](https://togithub.com/terraform-linters/tflint/pull/2073) - docs: Recommend verification with GitHub CLI by [@​wata727](https://togithub.com/wata727) in [https://github.com/terraform-linters/tflint/pull/2074](https://togithub.com/terraform-linters/tflint/pull/2074) **Full Changelog**: https://github.com/terraform-linters/tflint/compare/v0.51.1...v0.51.2

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about these updates again.



This PR has been generated by Mend Renovate. View repository job log here.

github-actions[bot] commented 3 months ago

🦙 MegaLinter status: ⚠️ WARNING

Descriptor Linter Files Fixed Errors Elapsed time
✅ API spectral 2 0 3.12s
✅ BASH bash-exec 5 0 0.02s
✅ BASH shellcheck 5 0 0.16s
✅ BASH shfmt 5 0 0 0.56s
✅ COPYPASTE jscpd yes no 5.02s
✅ DOCKERFILE hadolint 129 0 13.75s
✅ JSON jsonlint 18 0 0.2s
✅ JSON v8r 20 0 29.41s
⚠️ MARKDOWN markdownlint 266 0 272 37.93s
✅ MARKDOWN markdown-table-formatter 266 0 0 133.24s
✅ OPENAPI spectral 2 0 2.98s
⚠️ PYTHON bandit 212 64 3.56s
✅ PYTHON black 212 0 0 5.76s
✅ PYTHON flake8 212 0 2.33s
✅ PYTHON isort 212 0 0 1.64s
✅ PYTHON mypy 212 0 18.68s
✅ PYTHON pylint 212 0 17.13s
✅ PYTHON ruff 212 0 0 0.98s
✅ REPOSITORY checkov yes no 42.6s
✅ REPOSITORY git_diff yes no 0.6s
⚠️ REPOSITORY grype yes 1 23.05s
✅ REPOSITORY secretlint yes no 12.96s
✅ REPOSITORY trivy yes no 22.4s
✅ REPOSITORY trivy-sbom yes no 1.38s
⚠️ REPOSITORY trufflehog yes 1 12.88s
✅ SPELL cspell 694 0 27.49s
⚠️ SPELL lychee 347 5 10.76s
✅ XML xmllint 3 0 0 0.57s
✅ YAML prettier 161 0 0 5.63s
✅ YAML v8r 102 0 97.58s
✅ YAML yamllint 162 0 2.19s

See detailed report in MegaLinter reports

_MegaLinter is graciously provided by OX Security_

echoix commented 3 months ago

Why is the help command for terraform-fmt fail now? The changes don't seem to contain a change that could affect this

nvuillam commented 3 months ago

@echoix according to the errors, it's linter-helps.json that is unparseable strange

echoix commented 3 months ago

Something wrongly escaped?

echoix commented 3 months ago

I'll make it retry

echoix commented 3 months ago

Its not the same failure: https://github.com/oxsecurity/megalinter/actions/runs/9723818407/job/26839360161#step:11:3158 is FAILED megalinter/tests/test_megalinter/linters/terraform_terraform_fmt_test.py::terraform_terraform_fmt_test::test_get_linter_help

and https://github.com/oxsecurity/megalinter/actions/runs/9725560216/job/26843028822?pr=3712#step:11:3822 is FAILED megalinter/tests/test_megalinter/linters/typescript_standard_test.py::typescript_standard_test::test_format_fix

What exactly does the tests do just to make sure? And what are the pytest fixtures used (and their scopes)? Do they mount to the same folders and some tests try to change the same files with more than one Megalinter docker running at the same time? Something like it is not really well isolated at the files mounted level?

nvuillam commented 3 months ago

get_linter_help just checks that we succeed to call the help of the linter cli

test_format_fix checks that the linter update files to format/fix the check is done using git diff, so sometimes it can randomly fail because of parallel testing