-
Notifications
You must be signed in to change notification settings - Fork 0
Conversation
Marked this as ready for review. The basic functionality is there, but the structure will change in following PRs (once we have the execution token and data retrieval implemented) |
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.
Looks good!
f78bc6f
to
ad0495b
Compare
(Rebased to refresh) |
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 feedback: let me know what you think?
ad0495b
to
28a4907
Compare
(Rebased to refresh) |
Co-authored-by: Pi Delport <[email protected]>
Co-authored-by: Pi Delport <[email protected]>
Co-authored-by: Pi Delport <[email protected]>
28a4907
to
30068b8
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.
@longtomjr: Feedback addressed: diff.
Feel free to merge if you're happy with it!
Thanks a lot for get getting this sorted! |
No description provided.