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

bump derive_builder dependency to v0.12 #230

Merged
merged 3 commits into from May 30, 2023
Merged

bump derive_builder dependency to v0.12 #230

merged 3 commits into from May 30, 2023

Conversation

decathorpe
Copy link
Contributor

Version 0.12.0 of the derive_builder crate introduces only a small breaking change which does not affect this crate.

I have also bumped the MSRV of cargo_metadata to 1.56.0 - I tried compiling this crate with the currently set 1.42.0, which fails (even before bumping derive_builder to v0.12) because some of this crate's dependencies have moved to the Rust 2021 edition (which was stabilized in Rust 1.56.0).

Some dependencies of this crate have migrated to the 2021 Edition,
which was only stabilized in Rust 1.56.0.
@oli-obk
Copy link
Owner

oli-obk commented May 26, 2023

sgtm. You need to address the clippy failures though :)

Maybe we should also move this crate to the 2021 edition

@decathorpe
Copy link
Contributor Author

The clippy warning should now be addressed. I'm not sure if bumping to the 2021 edition would be necessary, unless you plan to use some features that are 2021-or-later-only. But at least it won't necessitate bumping the MSVR.

@oli-obk oli-obk merged commit 9e0a516 into oli-obk:main May 30, 2023
6 checks passed
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.

None yet

2 participants