Should we automatically switch to JSONC or arm-template in a parameter file? (was: Add comment in parameter file marked as error)

[yeswps]

When add comment in parameter file, it mark as an error under Problems section.

Looks like it's not recognizing it's a ARM parameter file, rather treated it as a regular JSON file

[StephenWeatherford]

@yeswps Thanks for taking the time to enter this issue. We do recognize the file but for various reasons don't currently change the language ID for parameter files. You can get rid of the errors by manually changing the language id to "JSONC" (click on "JSON" in the status bar).

Leaving open to consider automatically changing to JSONC for parameter files.

[yeswps]

@StephenWeatherford I've tried to switch the format to JSON with comment, it works in current session, but if I close the JSON file and reopen it, it came back been recognized as plan JSON file.

[StephenWeatherford]

@yeswps Thanks, I had thought vs code remembered the setting, but apparently not once it's closed. In that case I think we should definitely add the ability to automatically convert to jsonc.

In the meantime, I suggest adding this to your settings file:

"files.associations": {
    "*.parameters.json": "jsonc",
    "*.params.json": "jsonc"
}

[jodavis]

We are leaning toward changing the language ID to JSONC for these files.

[StephenWeatherford]

https://github.com/microsoft/vscode-azurearmtools/tree/saw/language-server-dotnet-acquisition-changebacktojson