search

sunchao / parquet-rs

Apache Parquet implementation in Rust
Apache License 2.0
147 stars 20 forks source link

Update cargo dependencies #162

Closed sadikovi closed 6 years ago

sadikovi commented 6 years ago

This small PR updates dependencies to the latest versions.

sadikovi commented 6 years ago

@sunchao Do you think that we should use ~ to restrict versions in Cargo.toml (https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html)?

coveralls commented 6 years ago

Pull Request Test Coverage Report for Build 621

Totals Coverage Status
Change from base Build 619: 0.0%
Covered Lines: 12415
Relevant Lines: 12993
💛 - Coveralls
sunchao commented 6 years ago

Thanks @sadikovi . The PR looks good to me. Regarding the "tilde" versions, I think we may not need it for now as #160 may just be a Rust nightly upgrade issue. Also, I checked some other major Rust projects and they all use the caret versions.