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

Update rand and quickcheck dependencies #61

Merged
merged 3 commits into from
Oct 4, 2020

Conversation

lionel-faber
Copy link
Contributor

This PR includes the commits automatically created by @dependabot along with the fixes needed for the dependency version updates.

Resolves #60
Supersedes #58 and #59

@shnewto
Copy link
Owner

shnewto commented Oct 2, 2020

🤔 these failures look like they're coming from cargo-travis, the crate we're using for test coverage reports... I've been thinking abt migrating the logic in that .travis.yml to use tarpaulin instead, looks like that'll need to happen first 🤔

I can prob hop into that in the next few days, I imagine it'll end up looking a lot like the setup here https://github.com/shnewto/ttaw/blob/master/.travis.yml

@shnewto
Copy link
Owner

shnewto commented Oct 3, 2020

@lionel1704 I merged some changes to the CI script that will hopefully address the failures on this PR. Would you mind syncing / updating your branch so we can make sure that did the trick?

dependabot bot and others added 3 commits October 4, 2020 11:23
Updates the requirements on [rand](https://github.com/rust-random/rand) to permit the latest version.
- [Release notes](https://github.com/rust-random/rand/releases)
- [Changelog](https://github.com/rust-random/rand/blob/master/CHANGELOG.md)
- [Commits](rust-random/rand@0.3.17...0.7.3)

Signed-off-by: dependabot[bot] <support@github.com>
Updates the requirements on [quickcheck](https://github.com/BurntSushi/quickcheck) to permit the latest version.
- [Release notes](https://github.com/BurntSushi/quickcheck/releases)
- [Commits](BurntSushi/quickcheck@quickcheck_macros-0.4.1...0.9.2)

Signed-off-by: dependabot[bot] <support@github.com>
- This commit also fixes some clippy errors
@coveralls
Copy link

Coverage Status

Coverage increased (+0.08%) to 83.981% when pulling e265db3 on lionel1704:deps-update into a7e08bd on shnewto:master.

@lionel-faber
Copy link
Contributor Author

That's rebased and updated 👍 CI looks happy 🎉

@shnewto
Copy link
Owner

shnewto commented Oct 4, 2020

Awesome! Thank you!!

@shnewto shnewto merged commit 4ed092f into shnewto:master Oct 4, 2020
@lionel-faber lionel-faber deleted the deps-update branch October 4, 2020 06:27
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

Successfully merging this pull request may close these issues.

Manually update dependabot's recommendations
3 participants