Add play framework configuration snippet #863
Conversation
| "mainClass": "play.core.server.ProdServerStart", | ||
| "args": [], | ||
| "jvmOptions": [], | ||
| "buildTarget": "???" |
There was a problem hiding this comment.
That's the default value that we use in the snippets. It would be better to allow the users choose when creating the snippet, but it's a bit more complicated.
There was a problem hiding this comment.
If only knowledge about build targets wasn't necessary.
There was a problem hiding this comment.
In this case it's necessary, since we don't know which target is a play server target :/
We could try to improve it for sure, but we don't keep the main classes from dependencies currently.
There was a problem hiding this comment.
Could we replace ??? with detailed information on how to fill this field? Like "Type build target name which has a dependency on play.core" or sth like this?
| @@ -726,7 +726,7 @@ | |||
| }, | |||
| "buildTarget": { | |||
| "type": "string", | |||
| "description": "Name of the build target of the class to run" | |||
| "description": "Name of the build target of the class to run, can be found in the projects section of the Metals tab" | |||
There was a problem hiding this comment.
I added a bit more info here, which gets displayed when you mouse over the build target. I feel like including the description in the value might be misleading.
No description provided.