trying to upgrade to version 2

[christophe-g]

Great to see snowpack evolving!

I am trying to upgrade one of my dependency (https://github.com/preignition/multi-chart) from version 1.7.1 to version 2.0.2

• pnpm add -D snowpack@latest [note: I use pnpm and npm or yarn - was working well with version 1.7.1]
• rename dedupe to installOptions.rollup.dedupe
• rename webDependencies to install

> pnpx snowpack install

(node:14940) ExperimentalWarning: The dns.promises API is experimental
(node:14940) ExperimentalWarning: The fs.promises API is experimental
ignoring unsupported file "coverage/lcov.info"
ignoring unsupported file "multi-chart.sublime-project"
ignoring unsupported file "multi-chart.sublime-workspace"

Error: Parse error @:92:1
    at Object.parse (/home/christophe/Programming/Polymer/litElement/multi-chart/node_modules/.pnpm/registry.npmjs.org/es-module-lexer/0.3.20/node_modules/es-module-lexer/dist/lexer.cjs:1:407)
    at getInstallTargetsForFile (/home/christophe/Programming/Polymer/litElement/multi-chart/node_modules/.pnpm/registry.npmjs.org/snowpack/2.0.2/node_modules/snowpack/dist-node/index.js:1195:35)
    at loadedFiles.filter.map.code (/home/christophe/Programming/Polymer/litElement/multi-chart/node_modules/.pnpm/registry.npmjs.org/snowpack/2.0.2/node_modules/snowpack/dist-node/index.js:1298:57)
    at Array.map (<anonymous>)
    at scanImports (/home/christophe/Programming/Polymer/litElement/multi-chart/node_modules/.pnpm/registry.npmjs.org/snowpack/2.0.2/node_modules/snowpack/dist-node/index.js:1298:45)

[christophe-g]

Solved by excluding some folders (in my case some .js files not properly formatted in a documentation folder).

It would be nice to have a more explicit error message, for instance which file is creating the parse error.

Migration remark
It took me some time to realize that version 2 scans the entire folder structure and fails to install to web_modules for this reason. For example, I had some unsolved dependencies (rollup-plugins) deep in a bower_components folder (yes, I am still using Polymer 2.0).

Running pnpx snowpack install against this repo was failing and complaining that rollup-plugin-node-polyfills or rollup-plugin-babel were not installed.

Thanks a lot for this new version anyway !

[FredKSchott]

That's a great call out, sounds like some action items to fix this are:

• be more explicit about what files are being scanned
• not fail completely on Parse error, just log that the file couldn't be scanned

[FredKSchott]

Thanks for taking the time to report. Spun off #369 to tackle, moving this discussion to our message boards

[pika-ci]

🚚 This issue has been moved!
Continue the discussion on our project message board:
https://www.pika.dev/npm/snowpack/discuss/249 • What is this?