Skip to content

Conversation

@al3mart
Copy link
Member

@al3mart al3mart commented Nov 14, 2025

These changes alleviate wait times giving feedback to the users after their calls have been included in a block such that they don't have to wait until the block their tx is included in has been finalized

@al3mart al3mart force-pushed the al3mart/fix-call-txstatus branch from 54dd342 to fa16e23 Compare November 14, 2025 17:20
Copy link
Collaborator

@moliholy moliholy left a comment

Choose a reason for hiding this comment

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

Looks good!

@codecov
Copy link

codecov bot commented Nov 14, 2025

Codecov Report

❌ Patch coverage is 0% with 34 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.19%. Comparing base (879ab31) to head (fa16e23).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
crates/pop-chains/src/call/mod.rs 0.00% 34 Missing ⚠️
@@            Coverage Diff             @@
##             main     #755      +/-   ##
==========================================
- Coverage   76.27%   76.19%   -0.09%     
==========================================
  Files         115      115              
  Lines       25991    26019      +28     
  Branches    25991    26019      +28     
==========================================
  Hits        19824    19824              
- Misses       4029     4057      +28     
  Partials     2138     2138              
Files with missing lines Coverage Δ
crates/pop-chains/src/call/mod.rs 54.41% <0.00%> (-8.66%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@AlexD10S AlexD10S merged commit bf25fb8 into main Nov 15, 2025
19 checks passed
@AlexD10S AlexD10S deleted the al3mart/fix-call-txstatus branch November 15, 2025 11:55
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.

4 participants