-Dvscode.workspace= should be removed about metals HOT 8 CLOSED

By the way, the language server communicates the workspace path at initialization. The only usage of the workspace directory before the server has started is for logging, but we can easily work around that. I think we should remove the workspace parameter entirely.

from metals.

I see, cool, then we should default to working directory .metaserver/*. We should keep the compilerconfig files there too IMO. Question if we should rename metaserver to something better before doing this?

from metals.

I think that's ok keeping the name for now. I don't think it will be such a breaking change if we rename it in the future.

from metals.

That's a good idea @laughedelic we should use the directories library to follow conventions of every OS. For the logs/.compilerconfig files we store it per-workspace.

from metals.

OK, cool, are y'all ok with using .metaserver/*? I think target/ is quite sbt specific and doesn't play nicely with other build tools.

from metals.

Sure, I think .metaserver is good enough for the time being.

from metals.

Do you mean ~/.metaserver? I thinks it's ok, but I would also suggest to follow the same way as with cache directories, i.e. use XDG and other analogous conventions on each system.

from metals.

I think @olafurpg means <projectWorkspace>/.metaserver, since it contains project-specific stuff such as the server logs.

from metals.