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

style: switch instances of allow to expect #59

Merged
merged 2 commits into from
Sep 5, 2024
Merged

style: switch instances of allow to expect #59

merged 2 commits into from
Sep 5, 2024

Conversation

GregoryConrad
Copy link
Owner

Fixes #58

@GregoryConrad
Copy link
Owner Author

Note to future self:

This will fail for the next couple months (until Rust 1.81 stable is released). At that point, rerun the CI here and it should be green. Or maybe not, which means we need to update the MSRV to 1.81 (in Cargo.toml, README, GitHub ci files).

@GregoryConrad GregoryConrad merged commit 339de09 into main Sep 5, 2024
6 checks passed
@GregoryConrad GregoryConrad deleted the 58-fix branch September 5, 2024 21:33
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.

Switch instances of allow to expect and add reasons in all areas
1 participant