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

chore(starknet_committer_and_os_cli): moved block-hash logic into separate folder #3842

Conversation

amosStarkware
Copy link
Collaborator

No description provided.

@reviewable-StarkWare
Copy link

This change is Reviewable

@amosStarkware amosStarkware force-pushed the 01-30-chore_starknet_committer_and_os_cli_moved_block-hash_logic_into_separate_folder branch from d198e43 to 083eb6b Compare January 30, 2025 11:47
Copy link
Collaborator

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

:lgtm:

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

Copy link
Collaborator

@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: all files reviewed, 1 unresolved discussion (waiting on @amosStarkware)


a discussion (no related file):
need py side pr

@amosStarkware amosStarkware force-pushed the 01-29-chore_starknet_committer_and_os_cli_moved_committer_logic_into_separate_folder branch from af0c1bd to 9a83178 Compare February 2, 2025 11:19
@amosStarkware amosStarkware force-pushed the 01-30-chore_starknet_committer_and_os_cli_moved_block-hash_logic_into_separate_folder branch from 083eb6b to e1b860f Compare February 2, 2025 11:19
@amosStarkware amosStarkware force-pushed the 01-29-chore_starknet_committer_and_os_cli_moved_committer_logic_into_separate_folder branch from 9a83178 to da0e82a Compare February 2, 2025 12:31
@amosStarkware amosStarkware force-pushed the 01-30-chore_starknet_committer_and_os_cli_moved_block-hash_logic_into_separate_folder branch from e1b860f to 4b7e7b2 Compare February 2, 2025 12:31
@amosStarkware amosStarkware force-pushed the 01-29-chore_starknet_committer_and_os_cli_moved_committer_logic_into_separate_folder branch from da0e82a to c332cc6 Compare February 2, 2025 12:52
@amosStarkware amosStarkware force-pushed the 01-30-chore_starknet_committer_and_os_cli_moved_block-hash_logic_into_separate_folder branch from 4b7e7b2 to c7cf596 Compare February 2, 2025 12:52
Copy link
Collaborator

@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: :shipit: complete! all files reviewed, all discussions resolved (waiting on @amosStarkware)

@amosStarkware amosStarkware force-pushed the 01-29-chore_starknet_committer_and_os_cli_moved_committer_logic_into_separate_folder branch from c332cc6 to 34298e6 Compare February 3, 2025 14:27
@amosStarkware amosStarkware force-pushed the 01-30-chore_starknet_committer_and_os_cli_moved_block-hash_logic_into_separate_folder branch from c7cf596 to 113d8cb Compare February 3, 2025 14:27
@ArniStarkware ArniStarkware changed the base branch from 01-29-chore_starknet_committer_and_os_cli_moved_committer_logic_into_separate_folder to main February 3, 2025 14:48
Copy link

graphite-app bot commented Feb 3, 2025

Merge activity

  • Feb 3, 10:01 AM EST: Graphite disabled "merge when ready" on this PR due to: We hit a GitHub rate limit while trying to merge. If you are attempting to merge a large stack, breaking it up into smaller chunks may help..

@amosStarkware amosStarkware force-pushed the 01-30-chore_starknet_committer_and_os_cli_moved_block-hash_logic_into_separate_folder branch from 113d8cb to e305e99 Compare February 3, 2025 15:05
@amosStarkware amosStarkware added this pull request to the merge queue Feb 3, 2025
Merged via the queue into main with commit 7705fb7 Feb 3, 2025
12 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.

3 participants