Add option to return none on unknown union variants

markgrex commented 1 year ago

Before this PR

See https://github.com/palantir/conjure-python-client/issues/129

https://github.com/palantir/conjure-python-client/pull/150 added the option in conjure-python-client to return None when encountering unknown union variants, but conjure-python doesn't include this option in the conjure generated code.

After this PR

This PR:

Upgrades the conjure-python-client dependency to 2.8.0
Updates the code generating a Service to propagate the _return_none_for_unknown_union_types flag to the decode method. This flag can be passed by the caller to RequestsClient.create(..)

==COMMIT_MSG== Add option to return none on unknown union variants ==COMMIT_MSG==

Possible downsides?

changelog-app[bot] commented 1 year ago

Generate changelog in changelog-dir>`changelog/@unreleased`</changelog-dir

Type

- [ ] Feature - [x] Improvement - [ ] Fix - [ ] Break - [ ] Deprecation - [ ] Manual task - [ ] Migration

Description

Add option to return none on unknown union variants

**Check the box to generate changelog(s)** - [x] Generate changelog entry

svc-autorelease commented 1 year ago

Released 4.8.0

palantir / conjure-python