starknet_os: os resources test - add 3 more syscalls#14138
Conversation
PR SummaryMedium Risk Overview Measured OS step/builtin costs for Reviewed by Cursor Bugbot for commit edfe6a8. Bugbot is set up for automated code reviews on this repo. Configure here. |
0d44efb to
cd7ac82
Compare
818a408 to
e55a063
Compare
cd7ac82 to
72d6d62
Compare
e55a063 to
01423bf
Compare
72d6d62 to
ba9b395
Compare
01423bf to
5d943b5
Compare
Yoni-Starkware
left a comment
There was a problem hiding this comment.
@Yoni-Starkware reviewed 4 files and all commit messages, and made 1 comment.
Reviewable status: 4 of 6 files reviewed, 2 unresolved discussions (waiting on dorimedini-starkware).
crates/blockifier_test_utils/resources/feature_contracts/cairo1/os_resources_test_contract.cairo line 104 at r1 (raw file):
// get execution info syscall. get_execution_info_v2_syscall().unwrap_syscall();
Same OS impl though
Suggestion:
get_execution_info_v3_syscall()ba9b395 to
1f21bca
Compare
5d943b5 to
1510ab1
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 1510ab1. Configure here.
1510ab1 to
edfe6a8
Compare


No description provided.