feat: in scala/metals, isHttpEnabled should be true by default. Also fixed an inconsistency between template & installer

[tribbloid]

No description provided.

[tribbloid]

regarding this part:

  "programArgs": {
    "windows": "${workingDir}/metals.bat",
    "default": "${workingDir}/metals"
  },

should this part be double quoted, like the code before this PR? Why is it necessary?

[angelozerr]

Double quoted support path which have spaces

[angelozerr]

Why you did this change in this PR? What it is workingDir?

I suppose you want to reference metals file from the project dir, right?

If it that it should be $PROJECT_DIR$

[tribbloid]

sorry I was copying from installer.json nearby, turns out I forgot the most important part:

"properties": {
"workingDir" : "$USER_HOME$/.lsp4ij/lsp/metals"
},

will add it back ASAP

[angelozerr]

Have you tried it? I fear that it will not work, you need to diplicate this HOME dir in default and windows command

[tribbloid]

ahh you are right, properties are not supported in template, will change to:

"$USER_HOME$/.lsp4ij/lsp/metals/metals"

which totally works on Linux:

for other OS, give me some time

[angelozerr]

Please add feat: before your commit in order to it follows conventional commit. After that I think we can merge it

[angelozerr]

Thanks @tribbloid !