Deprecate "initializationOptions" in favor of "initialization_options"#2779
Deprecate "initializationOptions" in favor of "initialization_options"#2779jwortmann merged 3 commits intosublimelsp:mainfrom
Conversation
✅ Deploy Preview for sublime-lsp ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
That's gonna be quite some job: Though I think I have a script somewhere that checks out each lsp package, applies chosen modification and creates PR automatically. |
|
I noticed that many packages reference the "allOf": [
{
"$ref": "sublime://settings/LSP-plugin-base"
},
{
"properties": {
"initialization_options": {
// ...
},
"settings": {
// ...
}
}
}
]but apparently that works fine, and I updated the schemas files accordingly in the packages that were created by me. Ideally all helper packages should reference And for packages which currently do use |
|
You want to create a definition for plugin settings (by convention called |
|
I was just casually going through the code, noticed one spot Line 367 in 42ff4ee
|
Use the snake case convention for settings in
*.sublime-settingsfiles, and make it consistent with all other configuration keys in the Client Configuration.This change should be fully backwards compatible with the old "initializationOptions", both for shipped settings in helper packages, and also for user overrides.
Ideally we should check which helper packages uses "initializationOptions" and update those, as well as the
sublime-package.jsonschema files.