Skip to content
This repository has been archived by the owner on Jan 9, 2025. It is now read-only.

Add casting utils #840

Merged
merged 4 commits into from
Dec 5, 2023
Merged

Conversation

ClementWalter
Copy link
Member

Time spent on this PR: 2

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

Some casting utils in utils.utils.Helpers, sometimes redundant, sometimes with non consistant naming, sometime very expensive.

Resolves #835

What is the new behavior?

New utils in dedicated files with EVM consistent naming and signature.

Copy link

codecov bot commented Dec 5, 2023

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (8d47fc4) 71.61% compared to head (1f76a28) 72.41%.
Report is 2 commits behind head on main.

Files Patch % Lines
src/utils/rlp.cairo 66.66% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #840      +/-   ##
==========================================
+ Coverage   71.61%   72.41%   +0.79%     
==========================================
  Files          49       49              
  Lines        6116     6043      -73     
==========================================
- Hits         4380     4376       -4     
+ Misses       1736     1667      -69     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Eikix
Eikix previously approved these changes Dec 5, 2023
Copy link
Member

@Eikix Eikix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Eikix
Eikix previously approved these changes Dec 5, 2023
Copy link
Member

@Eikix Eikix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@ClementWalter ClementWalter force-pushed the cw/casting-utils branch 2 times, most recently from aaf6d17 to 1f76a28 Compare December 5, 2023 15:29
@ClementWalter ClementWalter merged commit ce5d5ef into kkrt-labs:main Dec 5, 2023
7 checks passed
ClementWalter added a commit that referenced this pull request Dec 5, 2023
Following partial #840 merged after CI went ✅
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

stop unsafe casting from u256 to felt/address/u128
2 participants