Skip to content

Conversation

@keelerm84
Copy link
Member

This SDK already supports ldvalue.Optional(Int|Boolean|String) types.
However, this exposes a much more generic type that can be used without
those restrictions.

@keelerm84 keelerm84 requested a review from a team as a code owner May 19, 2025 16:18
This SDK already supports ldvalue.Optional(Int|Boolean|String) types.
However, this exposes a much more generic type that can be used without
those restrictions.
@keelerm84 keelerm84 force-pushed the mk/sdk-1258/option branch from ca8bb18 to 1c849fd Compare May 19, 2025 16:20
Copy link
Member

@kinyoklion kinyoklion left a comment

Choose a reason for hiding this comment

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

May want to consider a toPointer method.

@keelerm84 keelerm84 merged commit 1e748d5 into v3 May 19, 2025
16 checks passed
@keelerm84 keelerm84 deleted the mk/sdk-1258/option branch May 19, 2025 16:56
@github-actions github-actions bot mentioned this pull request May 19, 2025
keelerm84 pushed a commit that referenced this pull request May 19, 2025
🤖 I have created a release *beep* *boop*
---


##
[3.3.0](v3.2.0...v3.3.0)
(2025-05-19)


### Features

* Create generic Option type inspired by Rust
([#34](#34))
([1e748d5](1e748d5))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

3 participants