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

Relax required Terraform version to >= 0.12.0 #18

Closed
wants to merge 1 commit into from
Closed

Relax required Terraform version to >= 0.12.0 #18

wants to merge 1 commit into from

Conversation

anarsen
Copy link

@anarsen anarsen commented Dec 3, 2020

This enables use of Terraform 0.14 which was recently released.

Requires this to be merged and released first: cloudposse/terraform-aws-route53-cluster-hostname#29

@anarsen anarsen requested a review from a team as a code owner December 3, 2020 08:58
RothAndrew
RothAndrew previously approved these changes Dec 3, 2020
@anarsen
Copy link
Author

anarsen commented Dec 3, 2020

Thanks. Note that this PR needs further version bump on module dependencies, namely due to this: cloudposse/terraform-aws-route53-cluster-hostname#29

Marking this as draft to avoid accidental merging.

@anarsen anarsen marked this pull request as draft December 3, 2020 18:32
@RothAndrew
Copy link

/test all

@RothAndrew
Copy link

/rebuild-readme

This enables use of Terraform 0.14 which was recently released.

I wasn't sure whether you'd then like a `>= 0.12.0, < 0.15` constraint
instead. This is taken from
https://github.com/cloudposse/terraform-aws-acm-request-certificate/blob/master/versions.tf.
@anarsen anarsen marked this pull request as ready for review December 7, 2020 10:27
@anarsen
Copy link
Author

anarsen commented Dec 7, 2020

@RothAndrew Should be ready now. I bumped the version on the dependency towards terraform-aws-route53-cluster-hostname.

@joe-niland
Copy link
Member

/test all

@RothAndrew
Copy link

/test all

@anarsen
Copy link
Author

anarsen commented Dec 10, 2020

@RothAndrew Is there anything missing before this can be merged? I noticed the checks are failing, but the error isn't clear to me.

@RothAndrew
Copy link

https://sweetops.slack.com/archives/CUJPCP1K6/p1607444832056400

image

@maximmi maximmi mentioned this pull request Dec 30, 2020
@anarsen
Copy link
Author

anarsen commented Jan 6, 2021

Closed in favor of #20

@anarsen anarsen closed this Jan 6, 2021
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.

3 participants