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

Misc chores #43

Merged
merged 4 commits into from
Feb 18, 2024
Merged

Misc chores #43

merged 4 commits into from
Feb 18, 2024

Conversation

shuhuiluo
Copy link
Collaborator

No description provided.

Various package versions have been updated in Cargo.lock and Cargo.toml files. Major changes include updating version of "uniswap-sdk-core" to "0.15.0" in Cargo.toml and corresponding changes in Cargo.lock. Also, checksums and dependencies for updated packages have been adjusted accordingly.
Redundant comment lines were removed from several function and method descriptions. Additionally, settings in rustfmt.toml have been modified to enforce formatting in doc comments and to prefer field initialization shorthand.
The import statements in prelude.rs were refactored. Instead of individually importing components from the `chains`, `constants`, and `utils` modules, these entire modules are now imported. The old import method was replaced to simplify and clean up the code.
The `Default` trait has been implemented for the `Fraction` and `Percent` structures. It was used in tests of the `compute_price_impact` and fractions modules to replace manual initialization of these types, making the code clearer and easier to maintain.
@shuhuiluo shuhuiluo requested a review from malik672 February 18, 2024 08:49
@shuhuiluo shuhuiluo self-assigned this Feb 18, 2024
@malik672 malik672 merged commit e9fe9fe into malik672:master Feb 18, 2024
2 checks passed
@shuhuiluo shuhuiluo deleted the misc branch February 18, 2024 08:54
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