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

[QT-621] cty: use a fork of go-cty that fixes a panic when decoding #112

Merged
merged 2 commits into from Oct 5, 2023
Merged

[QT-621] cty: use a fork of go-cty that fixes a panic when decoding #112

merged 2 commits into from Oct 5, 2023

Conversation

ryancragun
Copy link
Collaborator

Checklist

  • The commit message includes an explanation of the changes
  • Manual validation of the changes have been performed (if possible)
  • New or modified code has requisite test coverage (if possible)
  • I have performed a self-review of the changes
  • I have made necessary changes and/or pull requests for documentation
  • I have written useful comments in the code

@ryancragun ryancragun added the changelog/bug Fix for something that wasn't working. Will be included in "Bug Fixes" category in release notes. label Sep 28, 2023
@ryancragun ryancragun requested a review from a team as a code owner September 28, 2023 17:10
@ryancragun
[QT-621] cty: use a fork of go-cty that fixes a panic when decoding t…
63030d0 
…he flightplan

Signed-off-by: Ryan Cragun <me@ryan.ec>
@ryancragun
go-cty: use our fork of go-cty that fixes a panic
2978ecf 
Signed-off-by: Ryan Cragun <me@ryan.ec>
@ryancragun ryancragun force-pushed the go-cty-panic branch 2 times, most recently from 6d98564 to 2978ecf Compare October 5, 2023 16:00
@ryancragun ryancragun merged commit 6b43b98 into main Oct 5, 2023
5 checks passed
@ryancragun ryancragun deleted the go-cty-panic branch October 5, 2023 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rebwill rebwill rebwill approved these changes

Assignees
No one assigned
Labels
changelog/bug Fix for something that wasn't working. Will be included in "Bug Fixes" category in release notes.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@ryancragun @rebwill