Skip to content

Conversation

@forfudan
Copy link
Owner

This pull request includes several changes to the DeciMojo library, focusing on code cleanup, new test additions, and minor optimizations. The most important changes include the addition of a new test case, a fix in the Decimal struct, and the removal of unnecessary comment lines.

New test additions:

Test removal:

  • tests/test_conversions.mojo: Removed the entire file containing tests for Decimal conversion methods and split it into test_to_string and test_to_int.

Code optimizations:

  • src/decimojo/decimal.mojo: Fixed the to_float method in the Decimal struct to correctly use Float64(10) for division, ensuring proper floating-point arithmetic.

Documentation updates:

  • docs/todo.md: Marked the optimization task for the exp() function as completed.

@forfudan forfudan merged commit f098174 into main Mar 15, 2025
1 of 2 checks passed
@forfudan forfudan deleted the work branch March 15, 2025 21:41
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