Skip to content

Conversation

giladchase
Copy link
Contributor

Just MOVE, no inner changes.

@reviewable-StarkWare
Copy link

This change is Reviewable

Copy link
Contributor Author

giladchase commented Sep 3, 2025

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

Just MOVE, no inner changes.
Copy link
Collaborator

@TomerStarkware TomerStarkware left a comment

Choose a reason for hiding this comment

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

:lgtm:

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

Copy link
Collaborator

@orizi orizi left a comment

Choose a reason for hiding this comment

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

@orizi reviewed all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @giladchase)


corelib/src/test/byte_array_test.cairo line 5 at r1 (raw file):

// ========= Test-utils =========

fn compare_byte_array(

all of this can probably just be replaced with assert_eq! and using printable strings:
(substring of `abcdefghi...ABCD..0..9)

Copy link
Contributor Author

@giladchase giladchase 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 @orizi)


corelib/src/test/byte_array_test.cairo line 5 at r1 (raw file):

Previously, orizi wrote…

all of this can probably just be replaced with assert_eq! and using printable strings:
(substring of `abcdefghi...ABCD..0..9)

Added to the other stack, closing this one.

@giladchase giladchase closed this Sep 4, 2025
@giladchase giladchase deleted the gilad/09-03-refactor_byte_array_hoist_test_utils branch September 4, 2025 07:01
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.

4 participants