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

JSON configuration pre-validation #768

Closed
Unitech opened this issue Oct 10, 2014 · 5 comments
Closed

JSON configuration pre-validation #768

Unitech opened this issue Oct 10, 2014 · 5 comments

Comments

@Unitech
Copy link
Owner

Unitech commented Oct 10, 2014

  • TV4
@soyuka
Copy link
Collaborator

soyuka commented Oct 28, 2014

try { JSON.parse(file) } catch(e) { console.error('json is not valid') }

?

@Unitech
Copy link
Owner Author

Unitech commented Oct 28, 2014

More something like checking if the exec_mode is cluster_mode or fork_mode for example

@Tjatse
Copy link
Collaborator

Tjatse commented Nov 20, 2014

I really need this enhancement very much, 'Cause I am using PM2 as a task manager, and all the confs are rendered by system, PM2 is easy crash if there have invalid options, I'm going to fix this iss, does the assignee exist?

PLUS: which one of branches should I work on? master or development?

@rlidwka
Copy link
Collaborator

rlidwka commented Nov 22, 2014

I'm going to fix this iss, does the assignee exist?

@Tjatse , technically I was planning to do this (oh my god, was it a month ago?), but go ahead. I mean, if you need it, you probably know better how it should be done.

@Tjatse
Copy link
Collaborator

Tjatse commented Nov 23, 2014

@rlidwka being developed :p

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants