Skip to content

chore(starknet_os): move write_split_result #5162

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

Merged

Conversation

dorimedini-starkware
Copy link
Collaborator

No description provided.

@reviewable-StarkWare
Copy link

This change is Reviewable

@dorimedini-starkware dorimedini-starkware self-assigned this Mar 23, 2025
@dorimedini-starkware dorimedini-starkware marked this pull request as ready for review March 23, 2025 09:23
@dorimedini-starkware dorimedini-starkware force-pushed the 03-23-feat_starknet_os_integrate_the_split_function branch from 76f91ed to f6f495a Compare March 23, 2025 09:39
@dorimedini-starkware dorimedini-starkware force-pushed the 03-23-chore_starknet_os_move_write_split_result branch from 63d9062 to 62007a5 Compare March 23, 2025 09:39
@dorimedini-starkware dorimedini-starkware force-pushed the 03-23-feat_starknet_os_integrate_the_split_function branch from f6f495a to 488ae74 Compare March 23, 2025 09:46
@dorimedini-starkware dorimedini-starkware force-pushed the 03-23-chore_starknet_os_move_write_split_result branch from 62007a5 to 70c050b Compare March 23, 2025 09:46
@dorimedini-starkware dorimedini-starkware force-pushed the 03-23-feat_starknet_os_integrate_the_split_function branch from 488ae74 to 373e57b Compare March 23, 2025 11:15
@dorimedini-starkware dorimedini-starkware force-pushed the 03-23-chore_starknet_os_move_write_split_result branch from 70c050b to 2cab525 Compare March 23, 2025 11:15
Copy link
Contributor

@aner-starkware aner-starkware left a comment

Choose a reason for hiding this comment

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

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

@dorimedini-starkware dorimedini-starkware force-pushed the 03-23-feat_starknet_os_integrate_the_split_function branch from 373e57b to fcb53e0 Compare March 23, 2025 15:57
@dorimedini-starkware dorimedini-starkware force-pushed the 03-23-chore_starknet_os_move_write_split_result branch from 2cab525 to 90f3373 Compare March 23, 2025 15:57
Copy link
Contributor

@aner-starkware aner-starkware left a comment

Choose a reason for hiding this comment

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

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

@dorimedini-starkware dorimedini-starkware force-pushed the 03-23-feat_starknet_os_integrate_the_split_function branch from fcb53e0 to 0d12f7e Compare March 23, 2025 16:29
@dorimedini-starkware dorimedini-starkware force-pushed the 03-23-chore_starknet_os_move_write_split_result branch from 90f3373 to 3d2a704 Compare March 23, 2025 16:29
@dorimedini-starkware dorimedini-starkware force-pushed the 03-23-feat_starknet_os_integrate_the_split_function branch from 0d12f7e to 20fb8b2 Compare March 24, 2025 14:31
@dorimedini-starkware dorimedini-starkware force-pushed the 03-23-chore_starknet_os_move_write_split_result branch from 3d2a704 to 02f6a83 Compare March 24, 2025 14:31
@dorimedini-starkware dorimedini-starkware force-pushed the 03-23-feat_starknet_os_integrate_the_split_function branch from 20fb8b2 to a01814d Compare March 25, 2025 09:49
@dorimedini-starkware dorimedini-starkware force-pushed the 03-23-chore_starknet_os_move_write_split_result branch from 02f6a83 to 6c8fc0c Compare March 25, 2025 09:49
@dorimedini-starkware dorimedini-starkware force-pushed the 03-23-feat_starknet_os_integrate_the_split_function branch from a01814d to d165e45 Compare March 25, 2025 11:52
@dorimedini-starkware dorimedini-starkware force-pushed the 03-23-chore_starknet_os_move_write_split_result branch from 6c8fc0c to 80ee16b Compare March 25, 2025 11:52
@dorimedini-starkware dorimedini-starkware force-pushed the 03-23-feat_starknet_os_integrate_the_split_function branch from d165e45 to 0dfa4c9 Compare March 25, 2025 13:07
@dorimedini-starkware dorimedini-starkware force-pushed the 03-23-chore_starknet_os_move_write_split_result branch from 80ee16b to a29dfa0 Compare March 25, 2025 13:07
@dorimedini-starkware dorimedini-starkware force-pushed the 03-23-feat_starknet_os_integrate_the_split_function branch from 0dfa4c9 to de9e590 Compare March 25, 2025 21:30
@dorimedini-starkware dorimedini-starkware force-pushed the 03-23-chore_starknet_os_move_write_split_result branch from a29dfa0 to b0a369b Compare March 25, 2025 21:30
@dorimedini-starkware dorimedini-starkware changed the base branch from 03-23-feat_starknet_os_integrate_the_split_function to main March 26, 2025 12:58
@dorimedini-starkware dorimedini-starkware added this pull request to the merge queue Mar 26, 2025
Merged via the queue into main with commit 4946ba5 Mar 26, 2025
16 checks passed
idan-starkware pushed a commit that referenced this pull request Mar 26, 2025
* feat(starknet_os): integrate the split function

* refactor(starknet_os): split to bigint3

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

* chore(starknet_os): move write_split_result

---------

Signed-off-by: Dori Medini <dori@starkware.co>
@github-actions github-actions bot locked and limited conversation to collaborators Mar 28, 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.

3 participants