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
Add hot code reload #1419
Add hot code reload #1419
Conversation
ed54f78
to
3e6747c
Compare
63be625
to
595334f
Compare
It will need both upcoming versions of the debugger (for the implementation) and metals (for the sync with compilation) to be fully functional Also, before merging I'll need to change the metals server's version before which HCR would not work to give proper information |
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.
Thanks for the great work! It will be amazing for the users.
Let's wait until we update scala debug adapter properly in Bloop and sbt
021efd3
to
6168803
Compare
6168803
to
4a1c36e
Compare
cleaning up allow hcr from specific metals version auto save files before hcr
ae40b85
to
9885716
Compare
9885716
to
51c854d
Compare
51c854d
to
2081c31
Compare
@tgodzik this PR is ready to be reviewed and merged. It adds the lightning button in the debug tool bar to trigger a hot code reload. |
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!
Address this feature request, but for now, it should only works with sbt, I'll look to make it works with bloop too later. I adapted the implementation from vscode-java-debug to work with metals
The implementation on the debugger side is in this pr (not published yet)
There are a few inconsistencies left in my implementation on the VS Code extension:
scalaHotReloadOn
is not ticked, the button still appear (we need to tick and untick to make it disappear)