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

upgrade deps to latest versions #47

Closed
wants to merge 1 commit into from
Closed

Conversation

paxsonsa
Copy link
Contributor

@paxsonsa paxsonsa commented Dec 5, 2023

Simple update of dependencies to make adopting this version of the crate more accessible. I could only execute the tests on my nix systems but it appears the dependency APIs have not changed.

Hope this is helpful!

@paxsonsa paxsonsa changed the title upgrade reps to latest versions upgrade deps to latest versions Dec 6, 2023
@willstott101
Copy link
Contributor

Apologies for taking so long to take a look at this. My windows machine can't run the ipv6 tests but otherwise passes with these updates.

Looks like dependency updates have raised the minimum supported rust version from under us anyway, so these updates seem to be fine.

Any particular reason to change some of the dependencies to specify all three version parts?

@paxsonsa
Copy link
Contributor Author

No worries, I figured you were busy!

Any particular reason to change some of the dependencies to specify all three version parts?

No particular reason, I think that is the default behaviour of cargo update I think! I can remove the patch values if you would like?

@paxsonsa
Copy link
Contributor Author

paxsonsa commented Jan 1, 2024

Went ahead and amended my commits and removed those patch values

@willstott101
Copy link
Contributor

7a2ab3e seems to be identical to 05048c1 in contents.

I merged the original commit and made a tiny tweak before publishing as 0.8.0. Thanks!

@paxsonsa
Copy link
Contributor Author

paxsonsa commented Jan 8, 2024

Thanks!

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.

2 participants