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

feat(starknet_api): chain gas prices hash (#962) #993

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

yoavGrs
Copy link
Contributor

@yoavGrs yoavGrs commented Sep 24, 2024

feat(starknet_api): chain gas prices hash (#962)

feat: add publish crates script (#893)

feat: implement unpublished crates verification (#894)

test(mempool): add TransactionQueueContent[Builder] (#967)

refactor(blockifier): unavailable contract address error (#970)

feat(blockifier): replace TryFrom<&Path> for VC with ad-hoc function (#973)

Signed-off-by: Dori Medini dori@starkware.co

feat(consensus): add active_proposals to context

chore(concurrency): store the state diff instead of the writes (#800)

feat(consensus): shc handle sm proposal

feat(batcher): mock BlockBuilder for tests

feat(mempool_infra): add serde utils

commit-id:93bc459d

chore(mempool_infra): wrap serde operations with generic custom struct towards binary serde

commit-id:325d0501

chore(ci): tune boostraping cache (#949)

chore(infra): add bootstrap flow to the ci (#941)

fix(blockifier): l1 to l2 gas price ratio (#815)

  • fix(blockifier): l1 to l2 gas price ratio

  • fix(blockifier): revert l2_to_l1 conversion for backwards compatibility

Merge main-v0.13.2 into main (#981)

Co-Authored-By: Nadin Jbara 93648739+nadin-Starkware@users.noreply.github.com
Co-Authored-By: alon-dotan-starkware alon.dotan@starkware.co

fix(ci): fix merge_branches.py, no changed_files dir (#984)

Signed-off-by: Dori Medini dori@starkware.co

Merge remote-tracking branch 'origin/main-v0.13.2' into noa/merge-main-v0.13.2-into-main-1727178366

No conflicts in main-v0.13.2 -> main merge, this commit is for any change needed to pass the CI.

feat(blockifier): versioned constants builtin names no longer strings (#971)

Signed-off-by: Dori Medini dori@starkware.co

refactor(mempool): use TransactionQueueContent[Builder] in MempoolContent[Builder] (#977)

Merge pull request #986 from starkware-libs/noa/merge-main-v0.13.2-into-main-1727178366

Merge main-v0.13.2 into main (#986)

Co-Authored-By: Nadin Jbara 93648739+nadin-Starkware@users.noreply.github.com
Co-Authored-By: alon-dotan-starkware alon.dotan@starkware.co
Co-Authored-By: dorimedini-starkware dori@starkware.co

refactor(consensus): remove ConsensusContext::Block

fix(blockifier): enforce fee for allresourcebounds when l1 gas is 0 (#846)

  • fix(blockifier): l1 to l2 gas price ratio

  • fix(blockifier): revert l2_to_l1 conversion for backwards compatibility

  • fix(blockifier): enforce fee for allresourcebounds when l1 gas is 0

refactor(consensus): minor refactor for how the log and db dirs are created and passed (#880)

fix(blockifier): remove duplicated function (#992)

refactor(blockifier): test pre validate


This change is Reviewable

Copy link
Contributor Author

yoavGrs commented Sep 24, 2024

Copy link

codecov bot commented Sep 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.28%. Comparing base (b0cfe82) to head (0d6c832).
Report is 52 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #993      +/-   ##
==========================================
- Coverage   74.18%   70.28%   -3.91%     
==========================================
  Files         359       87     -272     
  Lines       36240    11119   -25121     
  Branches    36240    11119   -25121     
==========================================
- Hits        26886     7815   -19071     
+ Misses       7220     2916    -4304     
+ Partials     2134      388    -1746     
Flag Coverage Δ
70.28% <ø> (-3.91%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

Copy link
Collaborator

@dorimedini-starkware dorimedini-starkware left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @yoavGrs)

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.

2 participants