Skip to content

Conversation

@dorimedini-starkware
Copy link
Collaborator

No description provided.

@reviewable-StarkWare
Copy link

This change is Reviewable

Copy link
Collaborator Author

dorimedini-starkware commented Oct 11, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@dorimedini-starkware dorimedini-starkware force-pushed the 10-11-starknet_os_flow_tests_migrate_test_direct_execute_call branch from 9492f14 to 7162c6f Compare October 23, 2025 08:56
@dorimedini-starkware dorimedini-starkware force-pushed the 10-11-blockifier_test_utils_update_meta-tx_test_contract branch from a2b1fe5 to aa71839 Compare October 23, 2025 08:56
@dorimedini-starkware dorimedini-starkware force-pushed the 10-11-starknet_os_flow_tests_migrate_test_direct_execute_call branch from 7162c6f to 63f153b Compare November 13, 2025 11:48
@dorimedini-starkware dorimedini-starkware force-pushed the 10-11-blockifier_test_utils_update_meta-tx_test_contract branch from aa71839 to ad2ca4f Compare November 13, 2025 11:48
@dorimedini-starkware dorimedini-starkware force-pushed the 10-11-starknet_os_flow_tests_migrate_test_direct_execute_call branch from 63f153b to 1ab0396 Compare November 13, 2025 13:47
@dorimedini-starkware dorimedini-starkware force-pushed the 10-11-blockifier_test_utils_update_meta-tx_test_contract branch from ad2ca4f to eb0e51c Compare November 13, 2025 13:47
@avivg-starkware
Copy link
Contributor

crates/starknet_os/src/hints/hint_implementation/compiled_class/compiled_class_test.rs line 55 at r1 (raw file):

Previously, meship-starkware (Meshi Peled) wrote…

@avivg-starkware might be worth checking why reverting the error instead of unwrapping results in a bigger margin.

Looks like the contract is much longer, so I think it makes sense that the margin is larger

Copy link
Collaborator

@meship-starkware meship-starkware left a comment

Choose a reason for hiding this comment

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

Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @AvivYossef-starkware and @Yoni-Starkware)

@dorimedini-starkware dorimedini-starkware force-pushed the 10-11-starknet_os_flow_tests_migrate_test_direct_execute_call branch from 1ab0396 to 4d86f40 Compare November 20, 2025 10:48
@dorimedini-starkware dorimedini-starkware force-pushed the 10-11-blockifier_test_utils_update_meta-tx_test_contract branch from eb0e51c to 66d34a6 Compare November 20, 2025 10:48
@dorimedini-starkware dorimedini-starkware force-pushed the 10-11-starknet_os_flow_tests_migrate_test_direct_execute_call branch from 4d86f40 to da87e12 Compare November 26, 2025 06:21
@dorimedini-starkware dorimedini-starkware force-pushed the 10-11-blockifier_test_utils_update_meta-tx_test_contract branch from 66d34a6 to c7f57a8 Compare November 26, 2025 06:21
@dorimedini-starkware dorimedini-starkware force-pushed the 10-11-starknet_os_flow_tests_migrate_test_direct_execute_call branch from da87e12 to 85b5a5d Compare December 2, 2025 10:58
@dorimedini-starkware dorimedini-starkware force-pushed the 10-11-blockifier_test_utils_update_meta-tx_test_contract branch from c7f57a8 to a73f1c4 Compare December 2, 2025 10:58
Copy link
Collaborator Author

@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.

@dorimedini-starkware reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @AvivYossef-starkware and @Yoni-Starkware)

@dorimedini-starkware dorimedini-starkware force-pushed the 10-11-starknet_os_flow_tests_migrate_test_direct_execute_call branch from 85b5a5d to b22ac4c Compare December 2, 2025 11:57
@dorimedini-starkware dorimedini-starkware force-pushed the 10-11-blockifier_test_utils_update_meta-tx_test_contract branch from a73f1c4 to 2d01273 Compare December 2, 2025 11:57
@graphite-app graphite-app bot changed the base branch from 10-11-starknet_os_flow_tests_migrate_test_direct_execute_call to graphite-base/9552 December 2, 2025 15:34
@dorimedini-starkware dorimedini-starkware force-pushed the 10-11-blockifier_test_utils_update_meta-tx_test_contract branch from 2d01273 to 831cefc Compare December 2, 2025 16:22
@dorimedini-starkware dorimedini-starkware changed the base branch from graphite-base/9552 to main-v0.14.1-committer December 2, 2025 16:22
Copy link
Collaborator Author

@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.

@dorimedini-starkware reviewed all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @AvivYossef-starkware and @Yoni-Starkware)

@dorimedini-starkware dorimedini-starkware force-pushed the 10-11-blockifier_test_utils_update_meta-tx_test_contract branch from 831cefc to 7ebfd03 Compare December 2, 2025 16:23
@dorimedini-starkware dorimedini-starkware force-pushed the 10-11-blockifier_test_utils_update_meta-tx_test_contract branch from 7ebfd03 to 5e97819 Compare December 2, 2025 19:23
@dorimedini-starkware dorimedini-starkware added this pull request to the merge queue Dec 2, 2025
Merged via the queue into main-v0.14.1-committer with commit 9a994ba Dec 2, 2025
30 checks passed
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.

5 participants