Skip to content

Conversation

@dorimedini-starkware
Copy link
Collaborator

No description provided.

@reviewable-StarkWare
Copy link

This change is Reviewable

Copy link
Contributor

@yoavGrs yoavGrs 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: 0 of 1 files reviewed, 1 unresolved discussion (waiting on @dorimedini-starkware and @rotem-starkware)


crates/starknet_os_flow_tests/src/tests.rs line 2967 at r1 (raw file):

#[rstest]
#[tokio::test]
async fn test_deploy_no_ctor_contract() {

Doc the test, as in the python repo

@dorimedini-starkware dorimedini-starkware force-pushed the 12-08-starknet_os_flow_tests_migrate_test_ec_op_scenario branch from 605340d to 49db718 Compare December 15, 2025 13:24
@dorimedini-starkware dorimedini-starkware force-pushed the 12-08-starknet_os_flow_tests_migrate_test_deploy_no_ctor_contract_scenario branch 2 times, most recently from 01bb702 to fcdacbb Compare December 15, 2025 13:28
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.

Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion (waiting on @rotem-starkware and @yoavGrs)


crates/starknet_os_flow_tests/src/tests.rs line 2967 at r1 (raw file):

Previously, yoavGrs wrote…

Doc the test, as in the python repo

Done.

@dorimedini-starkware dorimedini-starkware force-pushed the 12-08-starknet_os_flow_tests_migrate_test_ec_op_scenario branch from 49db718 to 4577ae9 Compare December 16, 2025 08:14
@dorimedini-starkware dorimedini-starkware force-pushed the 12-08-starknet_os_flow_tests_migrate_test_deploy_no_ctor_contract_scenario branch from fcdacbb to e67b3c4 Compare December 16, 2025 08:14
Copy link
Contributor

@yoavGrs yoavGrs left a comment

Choose a reason for hiding this comment

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

:lgtm:

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

@dorimedini-starkware dorimedini-starkware changed the base branch from 12-08-starknet_os_flow_tests_migrate_test_ec_op_scenario to main-v0.14.1-committer December 16, 2025 09:07
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 r4, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @rotem-starkware)

@dorimedini-starkware dorimedini-starkware added this pull request to the merge queue Dec 16, 2025
Merged via the queue into main-v0.14.1-committer with commit 0e2b36b Dec 16, 2025
21 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Dec 18, 2025
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