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.

Sign up for GitHub

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

Jump to bottom

Implement num_traits::Inv #494

Closed
turion opened this issue Mar 15, 2022 · 1 comment · Fixed by #495
Closed

Implement num_traits::Inv #494

turion opened this issue Mar 15, 2022 · 1 comment · Fixed by #495

Comments

@turion
Copy link
Contributor

turion commented Mar 15, 2022

num_traits is already a dependency, and some traits from it like CheckedDiv are already implemented. So it makes sense to support Inv as well. I can submit a PR if you're interested.
@paupino
Copy link
Owner

paupino commented Mar 15, 2022

Makes sense!

turion pushed a commit to turion/rust-decimal that referenced this issue Mar 16, 2022
Implement Inv impl for Decimal (paupino#494)
23cfc44
@turion turion mentioned this issue Mar 16, 2022
Merged
turion pushed a commit to turion/rust-decimal that referenced this issue Mar 16, 2022
Implement Inv impl for Decimal (paupino#494)
aa50715
paupino pushed a commit that referenced this issue Mar 17, 2022
@turion
Inv implementation for Decimal (#495)
4b97ec4 
Fixes #494

Co-authored-by: Manuel Bärenz <m.baerenz@sonnen.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Implement Inv for Decimal turion/rust-decimal
2 participants
@turion @paupino