-
Notifications
You must be signed in to change notification settings - Fork 82
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
Accept process.stdin
as image --input
#334
Comments
Good idea! Would you be so kind to open a PR? Not sure yet when I find time for it. Thanks! |
@axe312ger would like to but i get a lot of |
@xerc the errors you refer to are for node v21. The project installs fine for me on OSX (on my M1) and Linux (in CI) for node v18 and v20. Maybe try again with node 18 or 20 :) |
look good @ diff --git package.json
"devDependencies": {
+ "@types/debug": "4.1.12",
+ "@types/fs-extra": "11.0.4", |
@xerc awesome. I'll add them to master right away! |
Okay, thats done as well. About the process.stdin -> I wanna do this soon, probably also redo some of the logic that reads files and change how meta data is extracted and passed through the plugins. |
good idea ; incl. the checks for |
sqip -i test.jpg
=>curl -q https://[..]/test.jpg | sqip
sqip/packages/sqip-cli/src/sqip-cli.ts
Lines 36 to 39 in 91f9928
sqip/packages/sqip-cli/src/sqip-cli.ts
Lines 167 to 170 in 91f9928
The text was updated successfully, but these errors were encountered: