Skip to content

feat: resolve deprecated warnings #16

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

Merged
merged 1 commit into from
Aug 9, 2025
Merged

feat: resolve deprecated warnings #16

merged 1 commit into from
Aug 9, 2025

Conversation

sgtoj
Copy link
Collaborator

@sgtoj sgtoj commented Aug 9, 2025

No description provided.

@sgtoj sgtoj force-pushed the dev branch 2 times, most recently from 7daf6ae to 45fcb80 Compare August 9, 2025 14:33
@sgtoj sgtoj requested a review from Copilot August 9, 2025 14:33
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR resolves deprecated warnings in Terraform configuration by updating deprecated syntax and patterns to use modern equivalents.

  • Replaces deprecated try(coalesce(...)) pattern with one() function for AWS data source references
  • Removes deprecated managed_policy_arns argument from IAM role and replaces with explicit policy attachment resource
  • Adds AWS partition data source to make ARN construction more robust across different AWS partitions

@sgtoj sgtoj merged commit 014ba59 into main Aug 9, 2025
3 checks passed
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.

1 participant