-
Notifications
You must be signed in to change notification settings - Fork 364
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
mise settings improvements #3301
Comments
looks like escape characters are added to the json output that shouldn't be there. I think in json we should return arrays but in toml it should probably return one string for the array delimited by how the settings get read in. Otherwise you wouldn't be able to take the output from this command and use it as input into another config file.
I considered doing that in the initial version but the output is a lot less friendly IMO. I think we should add this behind a |
So the {
"disable_backends": ["java"],
"experimental": true
} I agree that the mise settings ls --json-with-sources
{
"disable_backends": { "value": ["java"], "source": "~/.config/mise/config.toml" },
"experimental": { "value": true, "source": "~/.config/mise/config.toml" }
} |
I think that's a good idea. Makes it clear it's only for json |
mise settings
has been improved in the latest releases and now offersjson
output. Thank you!Would it be possible to make the following changes? (I can create PRs)
toml
and/orjson
output could include the source of the settings, like it does for the table output (I have a similar request formise env --json
in Additional json output for mise doctor and mise env #3012)For context, I am trying to display them in https://github.com/hverlin/mise-vscode:
I have also noticed that
python.default_packages_file
does not show in the default output, even it's different from the default value. I might use the default output frommise
going forward, but just curious why isn't it shown?The text was updated successfully, but these errors were encountered: