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

Update to most recent main #43

Open
wants to merge 521 commits into
base: replay
Choose a base branch
from
Open

Update to most recent main #43

wants to merge 521 commits into from

Conversation

FrancoGiachetta
Copy link

No description provided.

meship-starkware and others added 30 commits December 29, 2024 13:24
…are-libs#2404)

* refactor(starknet_gateway): gateway business logic towards bench test

* chore(starknet_gateway): create bench file for benchmark test
…e-libs#2845)

- Currently only implementing the simplest event to parse, the rest are
  coming up.
- removed two unused error variants: `FeltParseError`, `Serde`

Co-authored-by: Gilad Chase <[email protected]>
…bs#2824)

- No reason to run cargo on release in the ci, and this busts cache
- Cargo doc doesn't pick up the usual place we pass -Dwarnings, which is
  RUSTFLAGS, it needs a special flag RUSTDOCFLAGS (it cannot reference
  the usual RUSTFLAGS so we have to duplicate)

Co-Authored-By: Gilad Chase <[email protected]>
dorimedini-starkware and others added 26 commits January 15, 2025 10:06
* chore(starknet_integration_tests): delete deprecated fns

commit-id:d9a2ae58

* chore(starknet_infra_utils): move fn from integration test

commit-id:8c7534bd

* chore(starknet_infra_utils): move fn from integration test

commit-id:81ec4ffa

* chore(starknet_infra_utils): unify test util mods

commit-id:3470ac8e
…3208)

Previous implementation inside transaction manager didn't account for
validate, the current implementation is more general, and can in the
future perhaps be reused elsewhere after some changes are made.

Co-authored-by: Gilad Chase <[email protected]>
Also add Anvil to the CI: moonrepo cannot cache binaries installed from
git, so this is _uncached_ and currently takes ~3 minutes to install.
Therefore, added a path filter to only run this installation when there
are changes in L1 crates.

Co-authored-by: Gilad Chase <[email protected]>
Fixture also needs to run only on CI, which makes it less useful,
removing.

Co-authored-by: Gilad Chase <[email protected]>
@FrancoGiachetta FrancoGiachetta marked this pull request as ready for review January 17, 2025 20:27
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.