Skip to content

Conversation

@avivg-starkware
Copy link
Contributor

No description provided.

@reviewable-StarkWare
Copy link

This change is Reviewable

@avivg-starkware avivg-starkware marked this pull request as ready for review July 16, 2025 13:57
Copy link
Collaborator

@noaov1 noaov1 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 3 of 3 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @AvivYossef-starkware)

@avivg-starkware avivg-starkware force-pushed the avivg/add_migration_to_blob_parms branch from 437b117 to 2d36ed7 Compare July 17, 2025 07:07
@avivg-starkware avivg-starkware force-pushed the avivg/add_migration_to_central_objects branch from 1d616f9 to 20f1ec9 Compare July 17, 2025 07:07
@avivg-starkware avivg-starkware force-pushed the avivg/add_migration_to_blob_parms branch from 2d36ed7 to a81a402 Compare July 17, 2025 07:12
@avivg-starkware avivg-starkware force-pushed the avivg/add_migration_to_central_objects branch from 20f1ec9 to 52ff22e Compare July 17, 2025 07:12
@avivg-starkware avivg-starkware force-pushed the avivg/add_migration_to_blob_parms branch from a81a402 to f8df09a Compare July 17, 2025 07:15
@avivg-starkware avivg-starkware force-pushed the avivg/add_migration_to_central_objects branch from 52ff22e to 82bce81 Compare July 17, 2025 07:15
@avivg-starkware avivg-starkware force-pushed the avivg/add_migration_to_blob_parms branch 2 times, most recently from fdcfdaf to aabbd98 Compare July 17, 2025 08:07
@avivg-starkware avivg-starkware force-pushed the avivg/add_migration_to_central_objects branch from 82bce81 to 2f6aec9 Compare July 17, 2025 08:07
@avivg-starkware avivg-starkware force-pushed the avivg/add_migration_to_central_objects branch from 2f6aec9 to b02ebf3 Compare July 17, 2025 11:05
@avivg-starkware avivg-starkware force-pushed the avivg/add_migration_to_blob_parms branch from aabbd98 to 61fb621 Compare July 17, 2025 11:05
Copy link
Collaborator

@noaov1 noaov1 left a comment

Choose a reason for hiding this comment

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

Reviewed 2 of 3 files at r2, 2 of 2 files at r3, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @AvivYossef-starkware)

@avivg-starkware avivg-starkware changed the base branch from avivg/add_migration_to_central_objects to graphite-base/8049 July 17, 2025 13:23
@avivg-starkware avivg-starkware force-pushed the avivg/add_migration_to_blob_parms branch from 61fb621 to cd51347 Compare July 17, 2025 13:23
@avivg-starkware avivg-starkware changed the base branch from graphite-base/8049 to main July 17, 2025 13:23
Copy link
Collaborator

@noaov1 noaov1 left a comment

Choose a reason for hiding this comment

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

Reviewed 2 of 2 files at r4, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @AvivYossef-starkware)

@avivg-starkware avivg-starkware force-pushed the avivg/add_migration_to_blob_parms branch from cd51347 to 6be90eb Compare July 17, 2025 14:01
Copy link
Collaborator

@noaov1 noaov1 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 r5, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @AvivYossef-starkware)

@avivg-starkware avivg-starkware added this pull request to the merge queue Jul 17, 2025
Merged via the queue into main with commit 865b155 Jul 17, 2025
12 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jul 19, 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