Skip to content
This repository was archived by the owner on Aug 8, 2025. It is now read-only.

Conversation

gefjon
Copy link
Contributor

@gefjon gefjon commented Apr 25, 2025

This must have gotten missed when porting the Rust ref over to C#.

This must have gotten missed when porting the Rust ref over to C#.
@gefjon gefjon requested a review from kazimuth April 25, 2025 13:29
Copy link
Contributor

@kazimuth kazimuth left a comment

Choose a reason for hiding this comment

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

LGTM with fix

Co-authored-by: james gilles <[email protected]>
@gefjon gefjon enabled auto-merge (squash) May 7, 2025 21:41
@gefjon gefjon merged commit e92d256 into master Jul 14, 2025
6 checks passed
bfops pushed a commit that referenced this pull request Aug 5, 2025
* Fix example that was still rust-ey in C# ref

This must have gotten missed when porting the Rust ref over to C#.

* Read `.Value`

Co-authored-by: james gilles <[email protected]>

---------

Co-authored-by: james gilles <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants