Skip to content
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

DatePerhapsTime to Chrono::NaiveDate #118

Open
arichtman opened this issue Nov 7, 2024 · 2 comments
Open

DatePerhapsTime to Chrono::NaiveDate #118

arichtman opened this issue Nov 7, 2024 · 2 comments

Comments

@arichtman
Copy link
Contributor

Wondering if you'd accept an implementation of DatePerhapsTime into NaiveDate.
Up to you if you'd like it as an additional method on a trait or an impl of From for DatePerhapsTime.

@hoodie
Copy link
Owner

hoodie commented Nov 7, 2024

Sure, why not? What's the use case?

@arichtman
Copy link
Contributor Author

I'm parsing in calendar files and trying to reduce the granularity of events to just dates so I can construct a simplified calendar representation for optimizing holiday placement.

I was thinking if this crate is already tied to Chrono it might make sense to allow Into.

Here's what I have presently to handle this

https://github.com/arichtman/icalendar-leave-optimiser/blob/e2fff089c9e9a3e0985aa1823a7afb80c33af549/src/cal_opt.rs#L65

arichtman added a commit to arichtman/icalendar-rs that referenced this issue Nov 7, 2024
arichtman added a commit to arichtman/icalendar-rs that referenced this issue Nov 17, 2024
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

No branches or pull requests

2 participants