search

jonnyboyC / kos-language-server

A language server for the Kerboscript (kOS) a language for Kerbal Space Program
MIT License
27 stars 6 forks source link

[BUG] path function should allow zero arguments #106

Closed kylehoover closed 5 years ago

kylehoover commented 5 years ago

Describe the bug The path function expects 1 argument (a path or string), but the argument should be optional.

To Reproduce Steps to reproduce the behavior:

  1. Create a new .ks file
  2. Type path().
  3. Hover over the closing paren and the suggestion will show that the path function is expecting 1 argument

Expected behavior As per the documentation here calling path() with no arguments creates a Path for the current directory. There shouldn't be a suggestion that path is expecting 1 argument.

Screenshots kos-path-bug

Editor

jonnyboyC commented 5 years ago

Thanks for the issue! This will be an easy fix. Please post more inconsistencies you find. This process isn't automated yet so I'm sure there are more places where something like this may have occurred

kylehoover commented 5 years ago

Will do! Thanks for all of the time you put into this!

jonnyboyC commented 5 years ago

Fixed by #107