Skip to content

Conversation

b3nj5m1n
Copy link
Contributor

@b3nj5m1n b3nj5m1n commented Jan 8, 2023

I needed this function myself, and apparently it doesn't currently exist.

I don't see why you wouldn't want this function, but if there's some reason for it, feel free to close the PR.

Even though the change is trivial, I added an example to sanity check that everything was still working, if you don't want the extra example, let me know and I'll update the PR.

@mikaelmello
Copy link
Owner

Great point.

When I saw your PR I was surprised to see I hadn't written this function too. Turns out that I did, but it is called with_default and in hindsight not at all clear.

I'll merge your PR as-is and do some small changes after that, it will be in the next release!

@mikaelmello mikaelmello merged commit 9956ef1 into mikaelmello:main Jan 9, 2023
@mikaelmello
Copy link
Owner

0.5.3 has your feature :)

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.

2 participants