Skip to content

fix: Remove deprecated Terraform syntax#48

Open
RoseSecurity wants to merge 1 commit into
mainfrom
fix/tflint-deprecated-syntax
Open

fix: Remove deprecated Terraform syntax#48
RoseSecurity wants to merge 1 commit into
mainfrom
fix/tflint-deprecated-syntax

Conversation

@RoseSecurity

@RoseSecurity RoseSecurity commented Mar 11, 2026

Copy link
Copy Markdown

Why

Deprecated HCL syntax triggers linting warnings and will eventually be removed in future Terraform versions. Cleaning these up improves code quality, maintainability, and ensures forward compatibility.

What

Automated fixes applied via tflint --fix with the following rules enabled:

Rule Description Example
terraform_deprecated_interpolation Remove unnecessary interpolation wrappers "${var.foo}" → var.foo
terraform_deprecated_index Replace legacy dot-index syntax list.0 → list[0]
terraform_deprecated_lookup Replace deprecated lookup() calls lookup(map, key) → map[key]

References

@RoseSecurity
fix: Remove deprecated Terraform syntax
be23730 
Automated fix using tflint --fix to resolve deprecated HCL syntax:

- terraform_deprecated_interpolation: Replace "${var.x}" with var.x
- terraform_deprecated_index: Replace .0 with [0]
- terraform_deprecated_lookup: Replace deprecated lookup() usage
@coderabbitai

coderabbitai Bot commented Mar 11, 2026

Copy link
Copy Markdown

Warning

Rate limit exceeded

@RoseSecurity has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 23 minutes and 44 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: aae751d6-2c28-40b6-8a63-b37cc981b5e4

📥 Commits

Reviewing files that changed from the base of the PR and between 50afcd5 and be23730.

📒 Files selected for processing (3)
  • src/main.tf
  • src/policy_gitops.tf
  • src/policy_lambda-cicd.tf
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/tflint-deprecated-syntax

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@mergify mergify Bot requested review from a team March 11, 2026 19:38
@mergify mergify Bot added the triage Needs triage label Mar 11, 2026
@nitrocode nitrocode added patch A minor, backward compatible change and removed triage Needs triage labels Apr 23, 2026
@nitrocode nitrocode added this pull request to the merge queue Apr 23, 2026
@mergify mergify Bot requested a review from a team April 23, 2026 02:01
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 23, 2026
@RoseSecurity

RoseSecurity commented Apr 23, 2026

Copy link
Copy Markdown
Author

/terratest

@RoseSecurity RoseSecurity added this pull request to the merge queue Apr 23, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 23, 2026
@github-actions

github-actions Bot commented Apr 23, 2026

Copy link
Copy Markdown
Contributor

There are no real tests for this component. So we set terratest statuses to successful execution without running any tests

@mergify

mergify Bot commented Apr 30, 2026

Copy link
Copy Markdown

Heads up! This pull request looks stale. It will be closed soon, if there are no new commits. ⏳

@mergify mergify Bot added stale This PR has gone stale and removed stale This PR has gone stale labels Apr 30, 2026
@RoseSecurity RoseSecurity added this pull request to the merge queue Apr 30, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 30, 2026
@mergify

mergify Bot commented May 7, 2026

Copy link
Copy Markdown

Heads up! This pull request looks stale. It will be closed soon, if there are no new commits. ⏳

@mergify mergify Bot added stale This PR has gone stale and removed stale This PR has gone stale labels May 7, 2026
@mergify

mergify Bot commented May 14, 2026

Copy link
Copy Markdown

Heads up! This pull request looks stale. It will be closed soon, if there are no new commits. ⏳

@mergify mergify Bot added stale This PR has gone stale and removed stale This PR has gone stale labels May 14, 2026
@mergify

mergify Bot commented May 21, 2026

Copy link
Copy Markdown

Heads up! This pull request looks stale. It will be closed soon, if there are no new commits. ⏳

@mergify mergify Bot added stale This PR has gone stale and removed stale This PR has gone stale labels May 21, 2026
@mergify

mergify Bot commented May 28, 2026

Copy link
Copy Markdown

Heads up! This pull request looks stale. It will be closed soon, if there are no new commits. ⏳

@mergify mergify Bot added stale This PR has gone stale and removed stale This PR has gone stale labels May 28, 2026
@RoseSecurity RoseSecurity added this pull request to the merge queue Jun 1, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jun 1, 2026
@RoseSecurity RoseSecurity added this pull request to the merge queue Jun 2, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jun 2, 2026
@mergify

mergify Bot commented Jun 4, 2026

Copy link
Copy Markdown

Heads up! This pull request looks stale. It will be closed soon, if there are no new commits. ⏳

@mergify mergify Bot added stale This PR has gone stale and removed stale This PR has gone stale labels Jun 4, 2026
@mergify

mergify Bot commented Jun 11, 2026

Copy link
Copy Markdown

Heads up! This pull request looks stale. It will be closed soon, if there are no new commits. ⏳

@mergify mergify Bot added stale This PR has gone stale and removed stale This PR has gone stale labels Jun 11, 2026
@mergify

mergify Bot commented Jun 18, 2026

Copy link
Copy Markdown

Heads up! This pull request looks stale. It will be closed soon, if there are no new commits. ⏳

@mergify mergify Bot added stale This PR has gone stale and removed stale This PR has gone stale labels Jun 18, 2026
@kevcube kevcube added this pull request to the merge queue Jun 18, 2026
@mergify mergify Bot requested a review from a team June 18, 2026 18:49
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jun 18, 2026
@mergify

mergify Bot commented Jun 25, 2026

Copy link
Copy Markdown

Heads up! This pull request looks stale. It will be closed soon, if there are no new commits. ⏳

@mergify mergify Bot added stale This PR has gone stale and removed stale This PR has gone stale labels Jun 25, 2026
@mergify

mergify Bot commented Jul 2, 2026

Copy link
Copy Markdown

Heads up! This pull request looks stale. It will be closed soon, if there are no new commits. ⏳

@mergify mergify Bot added stale This PR has gone stale and removed stale This PR has gone stale labels Jul 2, 2026
@mergify

mergify Bot commented Jul 9, 2026

Copy link
Copy Markdown

Heads up! This pull request looks stale. It will be closed soon, if there are no new commits. ⏳

@mergify mergify Bot added stale This PR has gone stale and removed stale This PR has gone stale labels Jul 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kevcube kevcube kevcube approved these changes
@nitrocode nitrocode nitrocode approved these changes

Assignees

No one assigned

Labels

patch A minor, backward compatible change

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@RoseSecurity @kevcube @nitrocode