-
Notifications
You must be signed in to change notification settings - Fork 220
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 tcpDebug option and tracing #618
base: main
Are you sure you want to change the base?
Conversation
@fwcd this is for the log feature |
53b2527
to
2ea49b1
Compare
server/build.gradle.kts
Outdated
@@ -91,6 +92,7 @@ tasks.register<JavaExec>("debugRun") { | |||
standardInput = System.`in` | |||
|
|||
jvmArgs(debugArgs) | |||
args(listOf("--tcpServerPort", serverDebugPort, "--tcpDebug", "--tracingLog")) |
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.
Hm this changes the default from stdio, right? Does
./gradlew debugRun --args '--tcpServerPort ... --tcpDebug --tracingLog'
work? If so, I wouldn't change this.
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.
Seems not work.. Maybe I do not really know how to make gradle work.. I also try to use --args to pass argument to task.. but it never works
@@ -176,6 +192,6 @@ class KotlinLanguageServer( | |||
// Fixed in https://github.com/eclipse/lsp4j/commit/04b0c6112f0a94140e22b8b15bb5a90d5a0ed851 | |||
// Causes issue in lsp 0.15 | |||
override fun getNotebookDocumentService(): NotebookDocumentService? { | |||
return null; | |||
} | |||
return null; |
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.
Same here (though if we reformat this, we could remove the redundant semicolon and write the function as = null
directly)
@Parameter(names = ["--tracingLog"]) | ||
var tracingLog: Boolean = false |
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'd probably prefer making this a more general option for passing the log level if that's an option
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 see, I will make this part to pass string, and if is tracing, I will tracing the function class name
b5a976a
to
9035968
Compare
9035968
to
dc24041
Compare
Sorry.. seems after rebase.. the commit disappear.. |
9d89ac9
to
2387d91
Compare
this made me easier to debug the project
and fix some warning