Merge main-v0.13.2 into main

[dorimedini-starkware]

This change is 

[dorimedini-starkware]

Src Dst .github/workflows/blockifier_compiled_cairo.yml
Src Dst Cargo.lock
Src Dst Cargo.toml

[dorimedini-starkware]

Reviewed 4 of 11 files at r6, 2 of 4 files at r8, 71 of 74 files at r9, 7 of 7 files at r10, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @dorimedini-starkware)

[github-actions]

Benchmark movements:
full_committer_flow performance regressed!
full_committer_flow time: [47.343 ms 47.390 ms 47.442 ms]
change: [+1.1218% +1.3268% +1.5194%] (p = 0.00 < 0.05)
Performance has regressed.
Found 9 outliers among 100 measurements (9.00%)
5 (5.00%) high mild
4 (4.00%) high severe

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.01%. Comparing base (5e5656b) to head (a0a04ce).
Report is 3 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #418   +/-   ##
=======================================
  Coverage   76.01%   76.01%           
=======================================
  Files         329      329           
  Lines       35296    35296           
  Branches    35296    35296           
=======================================
  Hits        26832    26832           
  Misses       6163     6163           
  Partials     2301     2301           

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

[amosStarkware]

Reviewed 3 of 3 files at r11, all commit messages.
Reviewable status: all files reviewed, 4 unresolved discussions (waiting on @dorimedini-starkware and @TzahiTaub)

---

Cargo.lock line 6287 at r10 (raw file):

<<<<<<< HEAD
||||||| 8134c27b
 "clap 4.5.4",

You don't want to specifiy the version?

Code quote:

"clap 4.5.4",

---

.github/workflows/blockifier_compiled_cairo.yml line 19 at r10 (raw file):

||||||| 8134c27b
=======
      - 'crates/blockifier/feature_contracts/cairo0/**'

Can you explain this? were your cairo1-compilation PRs over main?

Code quote:

 - 'crates/blockifier/feature_contracts/cairo0/**'

---

.github/workflows/blockifier_compiled_cairo.yml line 35 at r10 (raw file):

      - 'crates/blockifier/tests/requirements.txt'
||||||| 8134c27b
      - 'crates/blockifier/feature_contracts/cairo0/**'

ditto

Code quote:

- 'crates/blockifier/feature_contracts/cairo0/**'

---

.github/workflows/blockifier_compiled_cairo.yml line 95 at r10 (raw file):

          pip install -r crates/blockifier/tests/requirements.txt;
          cargo test -p blockifier --test feature_contracts_compatibility_test --features testing -- --include-ignored
>>>>>>> origin/main-v0.13.2

ditto

Code quote:

||||||| 8134c27b
      - run:
          pip install -r crates/blockifier/tests/requirements.txt;
          cargo test verify_feature_contracts -- --include-ignored
=======
      - run:
          pip install -r crates/blockifier/tests/requirements.txt;
          cargo test -p blockifier --test feature_contracts_compatibility_test --features testing -- --include-ignored
>>>>>>> origin/main-v0.13.2

[dorimedini-starkware]

Reviewed all commit messages.
Reviewable status: all files reviewed, 4 unresolved discussions (waiting on @amosStarkware and @TzahiTaub)

---

Cargo.lock line 6287 at r10 (raw file):

Previously, amosStarkware wrote…

You don't want to specifiy the version?

this version was only specified because the committer used a different version; we aligned the clap version all across the workspace so no more explicit version required.

---

.github/workflows/blockifier_compiled_cairo.yml line 19 at r10 (raw file):

Previously, amosStarkware wrote…

Can you explain this? were your cairo1-compilation PRs over main?

yes. they weren't needed on main-v0.13.2

---

.github/workflows/blockifier_compiled_cairo.yml line 35 at r10 (raw file):

Previously, amosStarkware wrote…

ditto

ditto

---

.github/workflows/blockifier_compiled_cairo.yml line 95 at r10 (raw file):

Previously, amosStarkware wrote…

ditto

the correct command is:

cargo test -p blockifier --test feature_contracts_compatibility_test --features testing -- --include-ignored

was fixed on both main and main-v0.13.2

[amosStarkware]

Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @TzahiTaub)

[amosStarkware]

Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @TzahiTaub)