-
Notifications
You must be signed in to change notification settings - Fork 93
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
Remove vm from simulator #796
Conversation
quote> vm package is removed. Few stuff has been moved into internal package
It deletes some functions signature and unused consts
@najeal going to take a little bit to get to this. I might also cause some conflicts. Sorry for the delay |
@richardpringle don't worry 🙂 |
This PR has become stale because it has been open for 30 days with no activity. Adding the |
@richardpringle Do you think I can resolve conflicts? |
@najeal, I'm going to close this one since there were so many changes. I'm going to say that we shouldn't do this right away (at least for now). Thanks again for the contribution! Even if it didn't land, the PR will be here if someone wants to take a look in the future and we all really appreciate the efforts! |
@richardpringle Don’t hesitate to ping me when you want it, I would be glad to make it 😁 |
This is related and closes #756
It deletes the
vm
package and moved the important stuff ininternal
.DB cleanup previously managed by the VM is now explicitly done in
cleanup
function of the simulator.