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

fix: pre-calculated addresses param #463

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

r4mmer
Copy link
Member

@r4mmer r4mmer commented Aug 5, 2024

Acceptance criteria

  • Check the correct key when using pre-calculated addresses argument.

@r4mmer r4mmer requested a review from pedroferreira1 as a code owner August 5, 2024 18:09
@r4mmer r4mmer requested a review from tuliomir August 5, 2024 18:09
@r4mmer r4mmer self-assigned this Aug 5, 2024
@r4mmer r4mmer requested review from alexruzenhack and removed request for pedroferreira1 August 5, 2024 18:09
Copy link

codecov bot commented Aug 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.07%. Comparing base (a1698c0) to head (caf3725).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #463      +/-   ##
==========================================
- Coverage   90.04%   88.07%   -1.98%     
==========================================
  Files          53       53              
  Lines        2280     2280              
  Branches      432      432              
==========================================
- Hits         2053     2008      -45     
- Misses        210      247      +37     
- Partials       17       25       +8     

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

3 participants