-
Notifications
You must be signed in to change notification settings - Fork 14
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
CSUB-358 Runtime-utils #828
Conversation
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.
Minor change requested, otherwise looks good to me.
704fc96
to
9249cd9
Compare
For full LLVM coverage report click here! |
Codecov Report
@@ Coverage Diff @@
## dev #828 +/- ##
==========================================
+ Coverage 78.06% 78.10% +0.03%
==========================================
Files 66 67 +1
Lines 10601 10608 +7
==========================================
+ Hits 8276 8285 +9
+ Misses 2325 2323 -2
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
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.
The nightly feature flag seems unnecessary (maybe a mistake?), but looks good otherwise
c1971ed
to
c55cc1d
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.
Needs a rebase to resolve conflicts, otherwise looks good to me.
8f84b10
to
232608b
Compare
* decouple extension builder * add runtime-utils and test client to dependabot * bump version * Auto-update creditcoin-js type definitions Co-authored-by: Carlos Gutierrez <cg@pop-os.localdomain> Co-authored-by: gluwa-bot <creditcoin@gluwa.com>
* decouple extension builder * add runtime-utils and test client to dependabot * bump version * Auto-update creditcoin-js type definitions Co-authored-by: Carlos Gutierrez <cg@pop-os.localdomain> Co-authored-by: gluwa-bot <creditcoin@gluwa.com>
* decouple extension builder * add runtime-utils and test client to dependabot * bump version * Auto-update creditcoin-js type definitions Co-authored-by: Carlos Gutierrez <cg@pop-os.localdomain> Co-authored-by: gluwa-bot <creditcoin@gluwa.com>
* decouple extension builder * add runtime-utils and test client to dependabot * bump version * Auto-update creditcoin-js type definitions Co-authored-by: Carlos Gutierrez <cg@pop-os.localdomain> Co-authored-by: gluwa-bot <creditcoin@gluwa.com>
* decouple extension builder * add runtime-utils and test client to dependabot * bump version * Auto-update creditcoin-js type definitions Co-authored-by: Carlos Gutierrez <cg@pop-os.localdomain> Co-authored-by: gluwa-bot <creditcoin@gluwa.com>
* decouple extension builder * add runtime-utils and test client to dependabot * bump version * Auto-update creditcoin-js type definitions Co-authored-by: Carlos Gutierrez <cg@pop-os.localdomain> Co-authored-by: gluwa-bot <creditcoin@gluwa.com>
* decouple extension builder * add runtime-utils and test client to dependabot * bump version * Auto-update creditcoin-js type definitions Co-authored-by: Carlos Gutierrez <cg@pop-os.localdomain> Co-authored-by: gluwa-bot <creditcoin@gluwa.com>
* decouple extension builder * add runtime-utils and test client to dependabot * bump version * Auto-update creditcoin-js type definitions Co-authored-by: Carlos Gutierrez <cg@pop-os.localdomain> Co-authored-by: gluwa-bot <creditcoin@gluwa.com>
* decouple extension builder * add runtime-utils and test client to dependabot * bump version * Auto-update creditcoin-js type definitions Co-authored-by: Carlos Gutierrez <cg@pop-os.localdomain> Co-authored-by: gluwa-bot <creditcoin@gluwa.com>
* decouple extension builder * add runtime-utils and test client to dependabot * bump version * Auto-update creditcoin-js type definitions Co-authored-by: Carlos Gutierrez <cg@pop-os.localdomain> Co-authored-by: gluwa-bot <creditcoin@gluwa.com>
* decouple extension builder * add runtime-utils and test client to dependabot * bump version * Auto-update creditcoin-js type definitions Co-authored-by: Carlos Gutierrez <cg@pop-os.localdomain> Co-authored-by: gluwa-bot <creditcoin@gluwa.com>
* decouple extension builder * add runtime-utils and test client to dependabot * bump version * Auto-update creditcoin-js type definitions Co-authored-by: Carlos Gutierrez <cg@pop-os.localdomain> Co-authored-by: gluwa-bot <creditcoin@gluwa.com>
* decouple extension builder * add runtime-utils and test client to dependabot * bump version * Auto-update creditcoin-js type definitions Co-authored-by: Carlos Gutierrez <cg@pop-os.localdomain> Co-authored-by: gluwa-bot <creditcoin@gluwa.com>
* decouple extension builder * add runtime-utils and test client to dependabot * bump version * Auto-update creditcoin-js type definitions Co-authored-by: Carlos Gutierrez <cg@pop-os.localdomain> Co-authored-by: gluwa-bot <creditcoin@gluwa.com>
* decouple extension builder * add runtime-utils and test client to dependabot * bump version * Auto-update creditcoin-js type definitions Co-authored-by: Carlos Gutierrez <cg@pop-os.localdomain> Co-authored-by: gluwa-bot <creditcoin@gluwa.com>
* decouple extension builder * add runtime-utils and test client to dependabot * bump version * Auto-update creditcoin-js type definitions Co-authored-by: Carlos Gutierrez <cg@pop-os.localdomain> Co-authored-by: gluwa-bot <creditcoin@gluwa.com>
* decouple extension builder * add runtime-utils and test client to dependabot * bump version * Auto-update creditcoin-js type definitions Co-authored-by: Carlos Gutierrez <cg@pop-os.localdomain> Co-authored-by: gluwa-bot <creditcoin@gluwa.com>
* decouple extension builder * add runtime-utils and test client to dependabot * bump version * Auto-update creditcoin-js type definitions Co-authored-by: Carlos Gutierrez <cg@pop-os.localdomain> Co-authored-by: gluwa-bot <creditcoin@gluwa.com>
* decouple extension builder * add runtime-utils and test client to dependabot * bump version * Auto-update creditcoin-js type definitions Co-authored-by: Carlos Gutierrez <cg@pop-os.localdomain> Co-authored-by: gluwa-bot <creditcoin@gluwa.com>
* decouple extension builder * add runtime-utils and test client to dependabot * bump version * Auto-update creditcoin-js type definitions Co-authored-by: Carlos Gutierrez <cg@pop-os.localdomain> Co-authored-by: gluwa-bot <creditcoin@gluwa.com>
* decouple extension builder * add runtime-utils and test client to dependabot * bump version * Auto-update creditcoin-js type definitions Co-authored-by: Carlos Gutierrez <cg@pop-os.localdomain> Co-authored-by: gluwa-bot <creditcoin@gluwa.com>
* decouple extension builder * add runtime-utils and test client to dependabot * bump version * Auto-update creditcoin-js type definitions Co-authored-by: Carlos Gutierrez <cg@pop-os.localdomain> Co-authored-by: gluwa-bot <creditcoin@gluwa.com>
Description of proposed changes:
Decouple externalities utils from offchain-task-scheduler. I picked offchain-task-scheduler instead of creditcoin's impl arbitrarily. I this PR passes we need to update Creditcoin's.
Practical tips for PR review & merge: