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

Replace nomnom with commander #117

Closed
wants to merge 1 commit into from

Conversation

cherewaty
Copy link

Recommended migration path from nomnom: https://github.com/harthur/nomnom/blob/master/README.md

Resolves #103

zaach/jison#367 also needs a merge to remove nomnom from the dependency chain.

@mbrookes
Copy link

mbrookes commented Oct 8, 2018

Could this be merged please? jsonlint (and therefore Material-UI) is getting flagged by tidelift.com / libraries.io due to this upstream dependency.

@FrontEndStudio
Copy link

nomnom: Package no longer supported. https://www.npmjs.com/package/nomnom

@mbrookes
Copy link

mbrookes commented Mar 13, 2019

@FrontEndStudio Judging by the lack of activity on this repo, it would seem jsonlint is also no longer supported (no commits, PRs merged, or issues responded to or closed in over a year).

@zaach If that's the case, it might be a good idea to flag this as unsupported in the README, deprecate on npm and archive the repo. Alternatively, there are quite a few forks showing recent activity. Perhaps one of those people would be happy to take over?

@eaviles
Copy link

eaviles commented Oct 17, 2019

@zaach any news? is there any way we can help here?

@mbrookes
Copy link

This appears to be the most active and well maintained fork: https://github.com/prantlf/jsonlint
cc @prantlf.

(I just don't understand why someone would abandon a library used in over 30,000 projects, rather than shut it down gracefully or hand it off to someone else to maintain.)

@cherewaty cherewaty closed this Dec 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

nomnom is deprecated
4 participants