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

Title: Update quoted type constraints to compliance with Terraform 0.15.3 #54

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bscriver123
Copy link

Pull Request Description

Description

This pull request addresses the issue of deprecated quoted references and type constraints in the AWS static site example, which is causing errors and warnings when running terraform init. The errors are related to usage of quoted references for type constraints, a behavior deprecated in versions later than Terraform 0.11. This PR updates the references to align with the new Terraform standards.

Changes Made

  • Removed the quoted references for type constraints in the following files:
    • .terraform/modules/my_site.aws_reverse_proxy/aws_reverse_proxy/variables.tf
    • .terraform/modules/my_site.aws_static_site/variables.tf

Resolution

Updated the type constraints to remove the quotes around "map" and updated to map(string) to explicitly indicate that the map elements are strings, as per Terraform latest version requirements.

Testing Done

Tested the changes locally by running terraform init to confirm that the errors related to invalid quoted type constraints no longer occur. Additionally, verified that the changes did not impact the functionality of the AWS static site deployment.

Version Info

  • Terraform v0.15.3
  • Provider registry.terraform.io/hashicorp/aws v3.40.0

This PR resolves the deprecation warnings and errors and ensures the AWS static site example is compatible with the latest Terraform standards.

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