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

Multiple files support #77

Open
ghost opened this issue Jan 2, 2016 · 9 comments
Open

Multiple files support #77

ghost opened this issue Jan 2, 2016 · 9 comments

Comments

@ghost
Copy link

ghost commented Jan 2, 2016

Hello,
I'd like to ask you to implement (and document) support for specifying multiple input files. That would allow us to run jsonlint from build (e.g. npm) scripts since we would like to pass the list of the files to lint from an environment variable. There is actually a lot of linters that accept multiple files already.

@ginman86
Copy link

ginman86 commented Jan 4, 2016

Checkout my fork - https://github.com/ginman86/jsonlint and PR - #75

@ghost
Copy link
Author

ghost commented Jan 5, 2016

Cool, great news!

@paladox
Copy link

paladox commented Jan 23, 2016

Hi could you open up the issue section and release it on npm please.

@terinjokes
Copy link

@ginman86 that looks great, I've switched to your branch. You should really start doing releases to npm

@sammdec
Copy link

sammdec commented Mar 2, 2016

What is the best way of getting this fork via npm?

@dawsbot
Copy link

dawsbot commented Mar 6, 2016

+1, this should go live on the main fork.

ssoloff added a commit to ssoloff/eslint-config-crockford that referenced this issue Nov 28, 2016
Note that the JSON Lint CLI only accepts a single file and does not
support globbing [1].  This required writing a custom NPM script to
perform the linting instead of invoking the jsonlint command directly.
This script can be replaced if multiple file support is added to JSON
Lint in the future.

[1] zaach/jsonlint#77
@AndersDJohnson
Copy link

Check out jsonlint-cli.

@prantlf
Copy link

prantlf commented May 19, 2019

I needed some bugfixes and extensions to jsonlint and did them in my fork. I released the changes as a new NPM module @prantlf/jsonlint. Accepting multiple files and/or directories on the command line was one of them. Otherwise I follow exactly the interface, parameters and naming conventions of the original package to make the switch easier.

@ssbarnea
Copy link

ssbarnea commented Jul 3, 2020

Something is fishy here. I do not see any reason why another tool would be needed for implementing the missing CLI behavior. Also the linked tool seems to have the last release more than 4 years ago.

This make me question the trust in both. Why having two less than ideally maintained tools instead of just one? ...especially as the ability to receive multiple file arguments seems to be a very basic feature of almost any linter.

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

No branches or pull requests

8 participants