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

fix: update version constraints for TPG v5 #187

Merged
merged 3 commits into from
Nov 6, 2023

Conversation

hollow
Copy link
Contributor

@hollow hollow commented Nov 6, 2023

Should be merged before #182 to fix TPG v5 support

@hollow hollow requested review from imrannayer and a team as code owners November 6, 2023 15:21
@hollow hollow changed the title Fix version constraints for TPG v5 fix: update version constraints for TPG v5 Nov 6, 2023
@hollow hollow changed the title fix: update version constraints for TPG v5 fix: update version constraints for TPG v5 Nov 6, 2023
@imrannayer
Copy link
Collaborator

@hollow you can remove version = "~> 4.0" from all the examples and let Modules dictate the version.

@@ -20,7 +20,7 @@ terraform {

google = {
source = "hashicorp/google"
version = ">= 4.42, < 5.0"
version = ">= 3.53, < 6"
Copy link
Collaborator

Choose a reason for hiding this comment

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

@hollow why r u changing min version to 3.53? Just change the max.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@imrannayer to align with the rest of the module. but I have changed it back to 4.42 now

@imrannayer
Copy link
Collaborator

/gcbrun

@imrannayer imrannayer self-requested a review November 6, 2023 17:02
@imrannayer imrannayer merged commit 63d105b into terraform-google-modules:master Nov 6, 2023
4 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.

2 participants