-
Notifications
You must be signed in to change notification settings - Fork 107
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
dev: replace blocking_write
with write
#552
dev: replace blocking_write
with write
#552
Conversation
blocking_write
with write
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #552 +/- ##
===========================================
+ Coverage 23.17% 72.76% +49.58%
===========================================
Files 9 45 +36
Lines 1247 4101 +2854
===========================================
+ Hits 289 2984 +2695
- Misses 958 1117 +159
☔ View full report in Codecov by Sentry. |
…ace-blockingwrite-with-write
…ace-blockingwrite-with-write
bc7ffbd
to
d4b6d36
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, will wait for @ClementWalter to approve
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
Time spent on this PR: 0.1 ## Pull request type Please check the type of change your PR introduces: - [x] Bugfix - [ ] Feature - [ ] Code style update (formatting, renaming) - [ ] Refactoring (no functional changes, no api changes) - [ ] Build related changes - [ ] Documentation content changes - [ ] Other (please describe): ## What is the current behavior? After removing the hardcoded Account for devnet, `get_account` was broken as testnet account uses `getPublicKey` case ## What is the new behavior? Fix `scripts.utils.get_account` for devnet
Time spent on this PR: 0.2
Resolves: #537
Pull Request type
Please check the type of change your PR introduces:
What is the new behavior?
Replace
blocking_write
used in Katana sequencer withwrite
Does this introduce a breaking change?