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.

Sign up for GitHub

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

Jump to bottom

Update to response body #16

Merged
merged 5 commits into from
Jul 27, 2022
Merged

Update to response body #16

merged 5 commits into from
Jul 27, 2022

Conversation

Benbentwo
Copy link
Member

@Benbentwo Benbentwo commented Jul 27, 2022 •
edited
Loading

what

  • Updates to use response_body
  • Updates versions.tf to matching versions

why

  • Latest release of HTTP Provider

references

@Benbentwo Benbentwo requested review from a team as code owners July 27, 2022 17:56
@Benbentwo
Copy link
Member Author

/test all

@Benbentwo
Copy link
Member Author

/test all

milldr
milldr approved these changes Jul 27, 2022
View reviewed changes
Copy link
Member

@milldr milldr left a comment

Choose a reason for hiding this comment

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

LGTM

@Benbentwo Benbentwo merged commit 41d2dfa into master Jul 27, 2022
@Benbentwo Benbentwo deleted the update_to_response_body branch July 27, 2022 18:17
@nitrocode
Copy link
Member

@Benbentwo @milldr how come the terraform version was bumped from 0.13 to 1.0 in this PR ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@milldr milldr milldr approved these changes

@dotCipher dotCipher Awaiting requested review from dotCipher dotCipher was automatically assigned from cloudposse/contributors

@korenyoni korenyoni Awaiting requested review from korenyoni korenyoni was automatically assigned from cloudposse/contributors

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Benbentwo @nitrocode @milldr @cloudpossebot