search

tmontaigu / dbase-rs

Rust library to read & write dBase files.
MIT License
29 stars 30 forks source link

replace chrono with time; fix some things clippy was upset about #35

Closed Roguelazer closed 2 years ago

Roguelazer commented 2 years ago

Fixes #34

This will require a major version bump.

If we want to keep compatibility for users who are stuck on chrono, I could imagine putting in a chrono feature flag that would gate the relevant impls.

tmontaigu commented 2 years ago

I merged it, time seems more active than chrono and has a bit more downloads so it's more used.

I yes if its requested we could put back the chrono conversion behind a feature flag

Thanks for the Pr