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

Change div_ceil to msrv_div_ceil #121

Merged
merged 1 commit into from
Mar 31, 2024
Merged

Change div_ceil to msrv_div_ceil #121

merged 1 commit into from
Mar 31, 2024

Conversation

kenken-neko
Copy link
Contributor

@kenken-neko kenken-neko commented Mar 31, 2024

Change

It has been modified to use msrv_div_ceil, which is implemented in the utils of this repository, instead of the div_ceil.

Reason

The div_ceil is unstable, which causes build errors such as the one in the image sent. To avoid this error.
divceil

Supplement

I have also confirmed that all tests of cargo test are completed without any problems.

@kenken-neko
Copy link
Contributor Author

kenken-neko commented Mar 31, 2024

Thank you for a very nice and beneficial implementation. I would appreciate it if you could review this PR when you have time.

@sarah-quinones
Copy link
Owner

lgtm!

@sarah-quinones sarah-quinones merged commit b014e06 into sarah-quinones:main Mar 31, 2024
@kenken-neko kenken-neko deleted the change_unstable_divceil_to_msrv branch March 31, 2024 11:26
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