-
Notifications
You must be signed in to change notification settings - Fork 30.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Valid settings not recognized in settings.json #199658
Comments
i was thinking on same lines just 2-3 days ago, that toml allows both formats it would drastically simplify my configuration as well. but the question is, does the > extCr = { "settings" : { "extensions.autoUpdate": false, "extensions.enableExperiments": false, } }
<- {settings: {…}}
> extProp = { "settings" : { "extensions": { "autoUpdate": false, "enableExperiments": false, } } }
<- {settings: {…}}
> extCr.settings.extensions
<- undefined
> extProp.settings.extensions
<- {autoUpdate: false, enableExperiments: false} so, these two i.e. so, i suppose the solution can be in either of these forms:
refences: |
it was plain collection of strings - i suppose all it does is take those strings, and output them in their own separate files - which would consume the backslashes, and produces the normal conclusion? - i didn't find anything fruitful in this regard in that file. here's its structure: {
"name": "",
"settings": "",
"keybindings": "",
"snippets": "",
"extensions": "",
"globalState": ""
}
2023-11-02
and it seems to be that vscode team is trying to avoid specifying nested objects, and rather escaping it & putting it as string. but i can't understand why? this makes it super hard to read. make the invesitgations multi-multi step & not toolable. and make tweaking & testing things up manually near impossible. |
I think this might be why #43670 (comment) def sort_settings_s(s):
true=True
false=False
f=lambda d: {k:f(d[k]) for k in sorted(list(d.keys()))} if isinstance(d,dict) else d
s2=f(eval(s))
return json.dumps(s2,indent=' '*4) |
We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider. If you wonder what we are up to, please see our roadmap and issue reporting guidelines. Thanks for your understanding, and happy coding! |
classic MS - our target audience is gooey using masses. though we love to name our tools as "power*" , but we dont like power users.
exactly what i was feeilng with the |
hey all! i have created a proposal which can help in grouping settings. though, not as properly as this, but still much better than no grouping. can u take a look? |
It would be nice to have a proper structure in settings.json. eg. to have this:
instead of this:
And it works both ways! The problem is these grouped settings are not recognized properly:
vscsettingsjson.mp4
Does this issue occur when all extensions are disabled?: Yes
Steps to Reproduce:
The text was updated successfully, but these errors were encountered: