-
-
Notifications
You must be signed in to change notification settings - Fork 192
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
feature request(cli): option to pass file containing filenames to lint #1166
Comments
note: #1167 would be an alternative solution if quick-lint respected |
I accept this task. |
Why not use |
So something like $ fd --extension=.js --extension=.jsx --extension=.ts --extension=.tsx | xargs -n999999999 quick-lint-js ? Because that makes too much sense. |
think of the microseconds saved from quick-lint being able to do it all itself! |
❯ echo "apps/us/e2e/full-regression/src/glass-orders/tc196-tiered -fast-refund -order.test.ts" | xargs -J{} quick-lint-js {}
error: unrecognized option: -fast-refund That's when I tried the other solutions I mentioned in the OP and forgot I had tried this! My bad |
@vegerot You should look at the |
Typical command: |
Problem:
I want to run quick-lint on a large project (120k files, 6m SLoC).
Approaches tried:
fd --extension=... | xargs -J{} quick-lint-js "{}"
doesn't work with filenames with spaces in themquick-lint-js $(fd ...)
zsh can't handle that many argumentsexec("/usr/local/bin/quick-lint-js", ...)
fails because it exceedsNCARGS
NCARGS
works but is inconvenient to usersRecommendation:
quick-lint CLI option that accepts the path to a file containing the paths of many files to check. This file can contain many more files than
NCARGS
.$ fd --extension=.js --extension=.mjs --extension=.jsx --extension=.ts --extension=.tsx > allfiles.txt $ quick-lint-js --file-with-filenames=allfiles.txt
Out of scope:
$ fd --extension=.js --extension=.mjs --extension=.jsx --extension=.ts --extension=.tsx | quick-lint-js --file-with-filenames=-
The text was updated successfully, but these errors were encountered: