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

Jump to bottom

CLI feature: validation #72

Closed
jordanbtucker opened this issue Jul 31, 2014 · 4 comments
Closed

CLI feature: validation #72

jordanbtucker opened this issue Jul 31, 2014 · 4 comments
Assignees
@jordanbtucker
Labels
enhancement 👍
Milestone
v1.0.0

Comments

@jordanbtucker
Copy link
Member

Current the CLI only converts JSON5 files to JSON. It validates the file in the process, so why not just have an option to validate your JSON5 files to see if they have any errors?
@aseemk
Copy link
Member

aseemk commented Aug 3, 2014

Nice idea. =)

@ravi
Copy link

ravi commented Oct 18, 2014

This is pretty critical, since, if there are errors during conversion the message provides no clue on the location of the error. (unless I am missing something :-))

@aseemk aseemk removed the needs-tests label Nov 5, 2014
@AVGP AVGP mentioned this issue Dec 23, 2014
Closed
@jordanbtucker jordanbtucker self-assigned this Oct 9, 2015
@jordanbtucker jordanbtucker added this to the v1.0.0 milestone Sep 20, 2017
@jordanbtucker
Copy link
Member Author

Partially fixed in 35269da

Still needs tests.

@jordanbtucker
Copy link
Member Author

Fixed in 51d0320.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jordanbtucker jordanbtucker

Labels
enhancement 👍
Projects
None yet
Milestone
v1.0.0
Development

No branches or pull requests
3 participants
@ravi @jordanbtucker @aseemk