-
Notifications
You must be signed in to change notification settings - Fork 63
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
feat: add machine emulator C API bindings #424
Conversation
This code has been extensively reviewed in the original PR. |
d6f99f6
to
3582cfe
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.
I only have a few comments about the CLI and the changelog for now, while the work is in progress.
4648bbd
to
e31192e
Compare
ab7ad59
to
4da0f72
Compare
This is ready to be merged after |
12f069d
to
5b38dac
Compare
2f773ff
to
a2ee630
Compare
5b38dac
to
e1b1e24
Compare
a2ee630
to
8f9f280
Compare
We can merge this now. |
8f9f280
to
46438d4
Compare
a65d62d
to
d60173a
Compare
96f8253
to
7cc7f11
Compare
Steps needed to install machine-emulator related dependencies should be part of this PR? |
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.
We should have kept the commit from the original author before applying our changes on top of it in order to separate our changes from his and to give him credit for what he did.
Besides that, I have added some suggestions for improvement that should be considered at least when this package is moved to its own repository.
Don't think so. In the future, we will use testcontainers or something like that. |
0e3d66a
to
a747e20
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.
@renan061 I have resolved all suggestions but we should create a ticket for the handover of these bindings to the machine team along with all suggested improvements
a747e20
to
8870048
Compare
From #335.