Skip to content
This repository has been archived by the owner on Jan 11, 2021. It is now read-only.

Update cargo dependencies #162

Merged
merged 1 commit into from
Sep 15, 2018
Merged

Conversation

sadikovi
Copy link
Collaborator

This small PR updates dependencies to the latest versions.

@sadikovi
Copy link
Collaborator Author

@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
Copy link

Pull Request Test Coverage Report for Build 621

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 95.551%

Totals Coverage Status
Change from base Build 619: 0.0%
Covered Lines: 12415
Relevant Lines: 12993

💛 - Coveralls

@sunchao
Copy link
Owner

sunchao commented Sep 14, 2018

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.

@sunchao sunchao merged commit a258939 into sunchao:master Sep 15, 2018
@sadikovi sadikovi deleted the cargo-update-deps branch September 15, 2018 08:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants