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.

Sign up for GitHub

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

Jump to bottom

Handle tflint deprecations #4451

Merged
merged 2 commits into from
Jul 9, 2023
Merged

Handle tflint deprecations #4451

merged 2 commits into from
Jul 9, 2023

Conversation

HorizonNet
Copy link
Contributor

@HorizonNet HorizonNet commented Jul 7, 2023 •
edited by Hanse00
Loading

Solve the deprecations introduced in tflint, which are getting removed in v0.47.0.

Unblock #4400

Proposed Changes

Change the command line call for tflint to remove the deprecations eliminated from tflint via this PR.

Readiness Checklist

Author/Contributor

  • [ x] If documentation is needed for this change, has that been included in this pull request

Reviewing Maintainer

  • Label as breaking if this is a large fundamental change
  • Label as either automation, bug, documentation, enhancement, infrastructure, or performance

@HorizonNet
Handle tflint deprecations
e095fa8 
Solve the deprecations introduced in tflint, which are getting removed
in v0.47.0.
@Hanse00 Hanse00 added the dependencies Pull requests that update a dependency file label Jul 9, 2023
@Hanse00
Copy link
Collaborator

Hanse00 commented Jul 9, 2023

Hi @HorizonNet, thanks for taking the time to contribute this!

For future reference, the exact tflint change that this relates to is: terraform-linters/tflint#1687

@Hanse00 Hanse00 self-assigned this Jul 9, 2023
@Hanse00 Hanse00 merged commit ec17c93 into super-linter:main Jul 9, 2023
4 checks passed
@HorizonNet HorizonNet deleted the tflint-deprecations branch July 10, 2023 08:38
@anthonyolo anthonyolo mentioned this pull request Jul 20, 2023
Closed
@terrorizer1980 terrorizer1980 mentioned this pull request May 13, 2024
Open
@aliscco aliscco mentioned this pull request May 13, 2024
Closed
@terrorizer1980 terrorizer1980 mentioned this pull request May 13, 2024
Closed
@Oleksandr1201vsp Oleksandr1201vsp mentioned this pull request May 14, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Hanse00 Hanse00 Hanse00 approved these changes

@admiralAwkbar admiralAwkbar Awaiting requested review from admiralAwkbar

@lindluni lindluni Awaiting requested review from lindluni

@zkoppert zkoppert Awaiting requested review from zkoppert zkoppert is a code owner

@nemchik nemchik Awaiting requested review from nemchik

@GaboFDC GaboFDC Awaiting requested review from GaboFDC

@ferrarimarco ferrarimarco Awaiting requested review from ferrarimarco ferrarimarco is a code owner

Assignees

@Hanse00 Hanse00

Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@HorizonNet @Hanse00