Skip to content
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

Request default of export to be --format=yaml #88

Closed
whowutwut opened this issue Aug 2, 2018 · 4 comments · Fixed by #98
Closed

Request default of export to be --format=yaml #88

whowutwut opened this issue Aug 2, 2018 · 4 comments · Fixed by #98
Labels
enhancement New feature or request sprint1
Milestone

Comments

@whowutwut
Copy link
Member

whowutwut commented Aug 2, 2018

Is there a benefit for using JSON over YAML?
If not, I think the files are more easily modified in source control if the default export format is YAML.

@robin2008
Copy link
Member

There is no special considering, as we want the inventory data model is a base of future xCAT, so json is more friendly for machine. For example, we canquick load the JSON data for Machine Learning.

And current, most the use case is better to use YAML, we can change the default to YAML.

@robin2008 robin2008 added this to the 2.14.3 milestone Aug 17, 2018
@robin2008
Copy link
Member

And I think no need to consider much about the impact after the default behavior changing as current install base is small.

@immarvin
Copy link
Contributor

yes, yaml is more friendly for human being , will consider this modification in next release. Since currently there is not many external customer, we do not need to consider backward compatibility

@immarvin
Copy link
Contributor

fixed in #98

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request sprint1
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants