Skip to content

Conversation

@moliholy
Copy link
Collaborator

Closes #738

This PR removes the --manual-weight flag, which is inconsistent with the pop up flow.

@moliholy moliholy self-assigned this Nov 14, 2025
@moliholy moliholy force-pushed the fix/remove-manual-weight branch 2 times, most recently from 58d1bf2 to 9778520 Compare November 14, 2025 13:20
@moliholy moliholy force-pushed the fix/remove-manual-weight branch from 9778520 to a6a247d Compare November 14, 2025 13:26
@codecov
Copy link

codecov bot commented Nov 14, 2025

Codecov Report

❌ Patch coverage is 81.25000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.32%. Comparing base (cf39bd3) to head (a6a247d).
⚠️ Report is 5 commits behind head on main.

Files with missing lines Patch % Lines
crates/pop-cli/src/commands/call/contract.rs 81.25% 1 Missing and 2 partials ⚠️
@@            Coverage Diff             @@
##             main     #752      +/-   ##
==========================================
- Coverage   76.33%   76.32%   -0.02%     
==========================================
  Files         115      115              
  Lines       25990    25973      -17     
  Branches    25990    25973      -17     
==========================================
- Hits        19840    19824      -16     
+ Misses       4010     4008       -2     
- Partials     2140     2141       +1     
Files with missing lines Coverage Δ
crates/pop-cli/src/commands/up/contract.rs 19.85% <ø> (ø)
crates/pop-cli/src/commands/call/contract.rs 72.72% <81.25%> (-0.43%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Collaborator

@AlexD10S AlexD10S left a comment

Choose a reason for hiding this comment

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

Thanks for removing this!

@moliholy moliholy merged commit 84e01ca into main Nov 14, 2025
19 checks passed
@moliholy moliholy deleted the fix/remove-manual-weight branch November 14, 2025 15:50
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.

pop up gas / proof size arguments inconsistent with pop call

3 participants