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 to toml_edit 0.22 #50

Closed
wants to merge 1 commit into from
Closed

Conversation

sdroege
Copy link

@sdroege sdroege commented Feb 6, 2024

No description provided.

@sdroege
Copy link
Author

sdroege commented Feb 6, 2024

Note that this time it does not involve a MSRV bump :)

@bkchr
Copy link
Owner

bkchr commented Feb 12, 2024

Doesn't look CI supports your statement 👀

@sdroege
Copy link
Author

sdroege commented Feb 13, 2024

Huh, did I fail reading and confused 7 with 9 😕 Sorry!

@fornwall
Copy link

fornwall commented Mar 8, 2024

What is the status here? Bumping MSRV is unfortunately still a mess, with different opinions and conventions.

But perhaps the MSRV should just be bumped here without a major version bump - those caring about being compatible with rust 1.68 and 1.69 can still version pin (it's for sure not an ideal situation, but perhaps the most practical)?

@torokati44
Copy link

Just a note about MSRV: Rust 1.69 is now almost as old as 1.67 was when this PR was submitted. This may or may not mean anything, but I think it's relevant. :)

@torokati44
Copy link

Bump...? 😶

@bkchr
Copy link
Owner

bkchr commented Aug 27, 2024

Done in: #52

@bkchr bkchr closed this Aug 27, 2024
@torokati44
Copy link

Yay, 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.

4 participants