-
Notifications
You must be signed in to change notification settings - Fork 6
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
refactor: poa fixes, tweaks, and chores #237
Conversation
Reecepbcups
commented
Dec 16, 2024
•
edited
Loading
edited
- Also migrates all int64 -> uint64 to just a sdkmath.NewIntFromUint64(uint64) consumer, rather than type converting. Makes linter happy :)
Simulation test failure is due to the new logic for denying duplicate ba78c0c Will fix later. Other CI 2 failures are unrelated. Main e2e & test run as expected |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #237 +/- ##
==========================================
- Coverage 60.01% 58.46% -1.56%
==========================================
Files 44 44
Lines 2116 2754 +638
==========================================
+ Hits 1270 1610 +340
- Misses 662 956 +294
- Partials 184 188 +4 ☔ View full report in Codecov by Sentry. |