Skip to content

Add from_partial for YearMonth/MonthDay FFI #351

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Manishearth
Copy link
Contributor

No description provided.

@nekevss
Copy link
Member

nekevss commented Jun 17, 2025

Just to confirm, have you seen #342? That's going to be a breaking change here to YearMonth::from_partial

@Manishearth
Copy link
Contributor Author

I have not, that is also an acceptable API. Still need it for MonthDay though. Happy to wait, I have an acceptable workaround for now.

@nekevss
Copy link
Member

nekevss commented Jun 17, 2025

I think I'd prefer to wait on this for now. Because there's also #349 to consider too.

@nekevss nekevss added blocked Waiting for some code change and removed blocked Waiting for some code change labels Jun 20, 2025
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