Skip to content
This repository has been archived by the owner on Jul 22, 2024. It is now read-only.

Fix fee calculation #709

Merged
merged 9 commits into from
Jul 3, 2023
Merged

Fix fee calculation #709

merged 9 commits into from
Jul 3, 2023

Conversation

matias-gonz
Copy link
Contributor

Add remaining gas to InvokeFunction execute call

@matias-gonz matias-gonz self-assigned this Jun 29, 2023
@matias-gonz matias-gonz marked this pull request as draft June 29, 2023 18:34
@matias-gonz matias-gonz changed the title Add remaining gas to InvokeFunction execute call Fix fee calculation Jun 30, 2023
@matias-gonz matias-gonz marked this pull request as ready for review June 30, 2023 15:21
@codecov-commenter
Copy link

codecov-commenter commented Jun 30, 2023

Codecov Report

Merging #709 (4b4e89e) into main (5dd7642) will increase coverage by 0.02%.
The diff coverage is 100.00%.

❗ Current head 4b4e89e differs from pull request most recent head 99616a3. Consider uploading reports for the commit 99616a3 to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #709      +/-   ##
==========================================
+ Coverage   92.36%   92.39%   +0.02%     
==========================================
  Files          52       52              
  Lines       11792    11819      +27     
==========================================
+ Hits        10892    10920      +28     
+ Misses        900      899       -1     
Impacted Files Coverage Δ
src/execution/execution_entry_point.rs 97.35% <100.00%> (ø)
src/lib.rs 98.56% <100.00%> (+0.05%) ⬆️
src/runner/mod.rs 80.97% <100.00%> (ø)
src/testing/mod.rs 100.00% <100.00%> (ø)
src/transaction/fee.rs 98.76% <100.00%> (ø)
src/transaction/mod.rs 88.09% <100.00%> (ø)

... and 2 files with indirect coverage changes

@juanbono juanbono added this pull request to the merge queue Jun 30, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Jun 30, 2023
@matias-gonz matias-gonz added this pull request to the merge queue Jul 3, 2023
Merged via the queue into main with commit ae35768 Jul 3, 2023
@juanbono juanbono deleted the fix_execute_invoke_function branch July 31, 2023 23:38
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.

4 participants