-
Notifications
You must be signed in to change notification settings - Fork 157
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
Use ByteArray
s instead of felt252
s in our own library assertions
#1892
Comments
5 tasks
github-merge-queue bot
pushed a commit
that referenced
this issue
Apr 10, 2024
…1999) <!-- Reference any GitHub issues resolved by this PR --> Closes #1892 ## Introduced changes <!-- A brief description of the changes --> - Use `ByteArray`s instead of `felt252`s in our own library assertions ## Checklist <!-- Make sure all of these are complete --> - [x] Linked relevant issue - [x] Updated relevant documentation - [x] Added relevant tests - [x] Performed self-review of the code - [x] Added changes to `CHANGELOG.md`
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Which components does the task require to be changed? (think hard pls)
snforge
Description
For example instead of
.expect('')
we can use longer error message and use bytearrayIn order to achieve that we can use macros to panic with bytearrays
The text was updated successfully, but these errors were encountered: