-
Notifications
You must be signed in to change notification settings - Fork 192
feat: add get_current_step getter #2034
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
Conversation
|
Benchmark Results for unmodified programs 🚀
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2034 +/- ##
=======================================
Coverage 96.54% 96.54%
=======================================
Files 102 102
Lines 42788 42791 +3
=======================================
+ Hits 41309 41312 +3
Misses 1479 1479 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Signed-off-by: Dori Medini <[email protected]>
Signed-off-by: Dori Medini <[email protected]>
Signed-off-by: Dori Medini <[email protected]>
7c8b3d3
to
48278c1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* feat: add get_current_step getter Signed-off-by: Dori Medini <[email protected]> * chore: changelog Signed-off-by: Dori Medini <[email protected]> * test: add coverage Signed-off-by: Dori Medini <[email protected]> --------- Signed-off-by: Dori Medini <[email protected]>
Add
get_current_step
getterDescription
To implement the OsLogger in the Starknet OS, access to the current step is required.
Checklist