Skip to content

Conversation

l-trotta
Copy link
Contributor

@l-trotta l-trotta commented Feb 11, 2025

Some variant tags can be boolean, like managed in ExplainLifecycleResponse. Adding boolean handling to json deserializers look ahead methods. Fixes #312, fixes #882 .

Everything works, except for the fact that the roundtrip doesn't work: "managed" gets ignored by deserialization.

Copy link
Member

@swallez swallez left a comment

Choose a reason for hiding this comment

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

LGTM

@l-trotta l-trotta merged commit f7017d8 into main Feb 17, 2025
8 checks passed
@l-trotta l-trotta deleted the lenient-tag branch February 17, 2025 16:35
github-actions bot pushed a commit that referenced this pull request Feb 17, 2025
* lifecycle response fix

* checkstyle

* update test

* cleaner code
github-actions bot pushed a commit that referenced this pull request Feb 17, 2025
* lifecycle response fix

* checkstyle

* update test

* cleaner code
github-actions bot pushed a commit that referenced this pull request Feb 17, 2025
* lifecycle response fix

* checkstyle

* update test

* cleaner code
l-trotta added a commit that referenced this pull request Feb 17, 2025
* lifecycle response fix

* checkstyle

* update test

* cleaner code

Co-authored-by: Laura Trotta <[email protected]>
l-trotta added a commit that referenced this pull request Feb 17, 2025
* lifecycle response fix

* checkstyle

* update test

* cleaner code

Co-authored-by: Laura Trotta <[email protected]>
l-trotta added a commit that referenced this pull request Feb 17, 2025
* lifecycle response fix

* checkstyle

* update test

* cleaner code

Co-authored-by: Laura Trotta <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

2 participants