search

hove-io / transit_model

Managing transit data with Rust
GNU Affero General Public License v3.0
57 stars 27 forks source link

feat: re-export external types from the API as part of the API #923

Open woshilapin opened 1 year ago

woshilapin commented 1 year ago

Today, all users of transit_model can't depend only on transit_model in their Cargo.toml because if one of the API returns a NaiveDate, or any Idx, the use basically needs to also import chrono and typed_index_collection. On top of that, it must find and import the exact same version of the dependency that the one declared in transit_model (or else, might face compilation errors).

Let's be more explicit on the API we expose. Note that I did not make an exhaustive research of all the things that we must re-export, only a brief dive to catch most of them. It's a step forward, might not be a perfect step though :shrug:

woshilapin commented 1 year ago

Fell free to move forward or close this PR. Since I won't be working on it anymore, I might close it if nothing happened at some point in the future.