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.

Sign up for GitHub

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

Jump to bottom

Consistent CLI options in dhall-json #1430

Closed
sjakobi opened this issue Oct 18, 2019 · 2 comments · Fixed by #1475
Closed

Consistent CLI options in dhall-json #1430

sjakobi opened this issue Oct 18, 2019 · 2 comments · Fixed by #1475
Labels
Hacktoberfest json/yaml

Comments

@sjakobi
Copy link
Collaborator

sjakobi commented Oct 18, 2019

#1414 introduced the --omissible-lists option for {json,yaml}-to-dhall. The same option should eventually replace --omitEmpty in dhall-to-{json,yaml}.

It would also be nice to get rid of camelCase in the dhall-json options. Let's rename --noMaps to --no-maps and --preserveNull to --preserve-null.
@astynax
Copy link
Contributor

astynax commented Oct 23, 2019

Can I take this one?

@sjakobi
Copy link
Collaborator Author

sjakobi commented Oct 23, 2019

@astynax That would be great! :)

If you have any questions, please ask right here!

@astynax astynax mentioned this issue Oct 25, 2019
Merged
@mergify mergify bot closed this as completed in #1475 Oct 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Hacktoberfest json/yaml
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Make some of CLI options for dhall-json more consistent
2 participants
@sjakobi @astynax