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
update for cstparser changes #238
Conversation
src/languageserverinstance.jl
Outdated
@@ -29,6 +29,8 @@ function send(message, server) | |||
end | |||
|
|||
function Base.run(server::LanguageServerInstance) | |||
io = open("/home/zac/jls.log", "a") |
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.
Absolute path?
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'll pull this change as we've got telemetry and potentially an upstream fix for the self-deleting logs
Codecov Report
@@ Coverage Diff @@
## master #238 +/- ##
==========================================
+ Coverage 38.67% 38.89% +0.21%
==========================================
Files 30 30
Lines 1872 1864 -8
==========================================
+ Hits 724 725 +1
+ Misses 1148 1139 -9
Continue to review full report at Codecov.
|
Tests pass, there's a backlog on travis that stopped them from completing though |
Updates for changes to CSTParser interface