Closed edzis closed 2 years ago
@edzis, hey there!
@343dev has fixed this behavior in #35. Would you mind to try again? 🐨
@igoradamenko @343dev There seems to be another issue:
➜ my-project git:(develop) optimizt ./src
node:internal/modules/cjs/loader:928
throw err;
^
Error: Cannot find module '../node_modules/svgo/lib/xast.js'
Require stack:
- /Users/X/.config/yarn/global/node_modules/@funboxteam/optimizt/svgo/removeUnknownsAndDefaults.js
- /Users/X/.config/yarn/global/node_modules/@funboxteam/optimizt/svgo/config.js
- /Users/X/.config/yarn/global/node_modules/@funboxteam/optimizt/lib/optimize.js
- /Users/X/.config/yarn/global/node_modules/@funboxteam/optimizt/index.js
- /Users/X/.config/yarn/global/node_modules/@funboxteam/optimizt/cli.js
at Function.Module._resolveFilename (node:internal/modules/cjs/loader:925:15)
at Function.Module._load (node:internal/modules/cjs/loader:769:27)
at Module.require (node:internal/modules/cjs/loader:997:19)
at require (node:internal/modules/cjs/helpers:92:18)
at Object.<anonymous> (/Users/X/.config/yarn/global/node_modules/@funboxteam/optimizt/svgo/removeUnknownsAndDefaults.js:4:45)
at Module._compile (node:internal/modules/cjs/loader:1108:14)
at Object.Module._extensions..js (node:internal/modules/cjs/loader:1137:10)
at Module.load (node:internal/modules/cjs/loader:973:32)
at Function.Module._load (node:internal/modules/cjs/loader:813:14)
at Module.require (node:internal/modules/cjs/loader:997:19) {
code: 'MODULE_NOT_FOUND',
requireStack: [
'/Users/X/.config/yarn/global/node_modules/@funboxteam/optimizt/svgo/removeUnknownsAndDefaults.js',
'/Users/X/.config/yarn/global/node_modules/@funboxteam/optimizt/svgo/config.js',
'/Users/X/.config/yarn/global/node_modules/@funboxteam/optimizt/lib/optimize.js',
'/Users/X/.config/yarn/global/node_modules/@funboxteam/optimizt/index.js',
'/Users/X/.config/yarn/global/node_modules/@funboxteam/optimizt/cli.js'
]
}
@edzis, it looks like the current behavior is somehow related to the difference between npm
and yarn
.
@343dev has probably fixed the error in #37. We've published the fix as @funboxteam/optimizt@2.7.5-rc. Would you mind to check it?
@edzis, woooosh! We've published one more RC version. Please, use this for the check: @funboxteam/optimizt@2.7.5-rc.2.
😅
👌 this works now and processes also the svg files! Thanks for your care!
I wish I had raised my concerns about require('../node_modules/svgo/...
earlier.
No problem. Thank you for the report!
We've finally published @funboxteam/optimizt@2.7.5 🎉
When the directory contains an svg file the CLI stops with the following error.
When I add filtering for errors via
const tasksResult = (await Promise.all(tasks)).filter(res => !!res);
inoptimize.js
I see the following output for those svg files:The SVG issue might be introduced in https://github.com/funbox/optimizt/commit/f4510cb139c047e4eb84a2e80915d4daca1b4d09.
But error filtering should be applied in any case.