Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • Oct 22 2018 18:48

    fwcd on master

    Added Gitter (compare)

Noah Santschi-Cooney
@Strum355
not super useful but thats besides the point :smile:
Noah Santschi-Cooney
@Strum355
what is currently being cached?
Noah Santschi-Cooney
@Strum355
any idea why hovering over a function name at its declaration site gives [ERROR : Hover]?
eg for fun hoverAt(file: CompiledFile, cursor: Int): Hover? { ...}
Noah Santschi-Cooney
@Strum355
okay i got a way to have function comments show up at where theyre called :smile: !
i shouldve looked at the code example you gave me, that wouldve pointed me to the right file to look at
Noah Santschi-Cooney
@Strum355
and it works
Noah Santschi-Cooney
@Strum355
Tests are failing for some reason in the PR, tried looking into it but i cant explain why
Noah Santschi-Cooney
@Strum355
trying to debug the failing tests, junit is so hard to deal with when trying to get info on why stuff is failing...
Noah Santschi-Cooney
@Strum355
okay i got a fix, need to understand why its needed tho
Noah Santschi-Cooney
@Strum355
return type of: CompletableFuture<List<DocumentHighlight>>, hover gives CompletableFuture<List<[ERROR : DocumentHighlight]>>... where does the ERROR come from and why is it there if it would be correct without it
F. W.
@fwcd_gitlab
Hmm, I have experienced some weird issues with generic types in hovers too
I will take a look at your PRs this weekend
Noah Santschi-Cooney
@Strum355
no worries, the two of them are still relatively WIP as in they work for certain cases but would ideally be extended for more cases sometime in the near or far future. Can merge them as they are if you wish
Noah Santschi-Cooney
@Strum355
java extension is able to give javadoc for built-in types (maybe for other types as well?) Ill look into how they get it over the next while
Noah Santschi-Cooney
@Strum355
weird error in a signature helper function, where it calculates the difference between the cursor and the start of the parameter list and uses that offset in getting a subsequence of the parameter list string. It causes a string out of bounds if the difference is larger than the length of the string, so i need to find out what the function is actually trying to do with its code
Noah Santschi-Cooney
@Strum355
been super busy with college, sorry theres been no work done
Noah Santschi-Cooney
@Strum355
semester over, back to kotlin
Tomer
@Tadaboody
:tada:
Joshua Cristol
@jcristol
hello i'm working on using Kotlin in vscode and i'm having trouble getting the extension Kotlin which uses the KotlinLanguageServer working.
I have installed a fresh version of vscode and the only extension I have installed is https://marketplace.visualstudio.com/items?itemName=fwcd.kotlin
I think the language server is launching given my vscode developer debug console but I don't really know where to go from here to understand why I am not getting code completion or syntax suggestions
Interestingly I am getting syntax highlighting could that be a sign of the language server partially working??
My spec are running Kotlin on OSX Mojave and I have kotlin installed through brew
Joshua Cristol
@jcristol
^ disregard I got everything to work by clearing out vscode's settings.json
F. W.
@fwcd_gitlab
That's great! Syntax highlighting uses a TextMate grammar file though and is not part of the language server. (neither is it supported by LSP) Since VSCode loads this grammar directly from the extension package, it will work indepedently.
Noah Santschi-Cooney
@Strum355
@fwcd_gitlab what symbols do you mean in fwcd/KotlinLanguageServer#10
Noah Santschi-Cooney
@Strum355
also
image.png
is this something youd like to have or would you prefer the original:
image.png
personally i prefer the top one as you can see the different overloads straight away at a glance without having to scroll through each to see one at a time
F. W.
@fwcd_gitlab
The top one looks good 👍 AFAIK the Java language server presents completions in a similar way

@fwcd_gitlab what symbols do you mean in fwcd/KotlinLanguageServer#10

Showing unimported items in the completion list and automatically importing them when the user selects the item. For instance, when typing File, code completion should automatically pop up and suggest java.io.File. Once the user hits return (or tab), java.io.File should be added to the imports at the beginning of the file.

Which would essentially mimic the behavior of the Java extension
F. W.
@fwcd_gitlab
As noted in the issue, I did already experiment with such a feature a while ago by querying the entire classpath (see this separate branch and fwcd/KotlinLanguageServer#10). That implementation, however, had some severe performance issues, but if you are willing to look into it, feel free 👍 This feature would make editing a lot more comfortable IMO
Noah Santschi-Cooney
@Strum355
oh i wasnt aware that feature doesnt exist already :thinking: thought it did
maybe im confusing it with it being in the java one
i can definitely have a look into that, if the only issue is performance then i should be able to help
Noah Santschi-Cooney
@Strum355
[Info  - 19:53:06] main      Connected to client
[Info  - 19:53:06] client    Adding workspace file:///home/noah/Kotlin/KotlinLanguageServer to source path
[Info  - 19:53:06] client    Adding 101 files under /home/noah/Kotlin/KotlinLanguageServer to source path
[Info  - 19:53:06] client    Searching for dependencies in workspace root /home/noah/Kotlin/KotlinLanguageServer
[Info  - 19:53:06] client    Attempting dependency resolution through CLI...
[Info  - 19:53:06] client    Creating temporary gradle file /tmp/classpath3454683406276641897.gradle
[Info  - 19:53:51] client    Successfully resolved dependencies using Gradle dependencies CLI
[Info  - 19:53:51] client    Attempting dependency resolution through CLI...
[Info  - 19:53:51] client    Creating temporary gradle file /tmp/classpath1518563813544658597.gradle
[Info  - 19:53:51] client    Found gradle at /usr/sbin/gradle
[Info  - 19:53:53] client    Successfully resolved dependencies using Gradle dependencies CLI
[Info  - 19:53:53] client    Attempting dependency resolution through CLI...
[Info  - 19:53:53] client    Creating temporary gradle file /tmp/classpath2782203026256202926.gradle
[Info  - 19:53:55] client    Successfully resolved dependencies using Gradle dependencies CLI
[Info  - 19:53:55] client    Attempting dependency resolution through CLI...
[Info  - 19:53:55] client    Creating temporary gradle file /tmp/classpath6875696104469076848.gradle
[Info  - 19:53:57] client    Successfully resolved dependencies using Gradle dependencies CLI
[Info  - 19:53:57] client    Adding 30 files to class path
the first temporary gradle file creation is really slow
why does it make multiple though?
F. W.
@fwcd_gitlab
I have noticed that too, but not got around debugging it (yet)
More verbose logging of Gradle's output might help spotting the issue
F. W.
@fwcd_gitlab
I have opened an issue for that: fwcd/KotlinLanguageServer#82
Noah Santschi-Cooney
@Strum355
nice one :ok_hand: Hows life been since? Still busy with school? :smile:
F. W.
@fwcd_gitlab
Studying comp sci and mathematics now, so yeah, pretty busy most of the time 😄
Noah Santschi-Cooney
@Strum355
Ah snap mathematics as well 🙃 tough going. Am in my 3rd year of comp sci, surprisingly busy semester. Ill be starting work placement in march so if all goes well ill have good time in the evenings to contribute more
Assuming im not dead tired 👀
Noah Santschi-Cooney
@Strum355
error TS6059: File '/home/noah/Kotlin/KotlinLanguageServer/bin/vscode-extension-src/extension.ts' is not under 'rootDir' '/home/noah/Kotlin/KotlinLanguageServer/vscode-extension-src'. 'rootDir' is expected to contain all source files.
getting these now since i freshly cloned : (