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

Error on 'npm install' - Unexpected key 'output' found #22

Closed
gavinr opened this issue Oct 3, 2019 · 1 comment
Closed

Error on 'npm install' - Unexpected key 'output' found #22

gavinr opened this issue Oct 3, 2019 · 1 comment

Comments

@gavinr
Copy link
Contributor

gavinr commented Oct 3, 2019

When I run npm install in this repository, I get an error on the build portion:

> [email protected] build C:\.....\esri-leaflet-heatmap
> rollup -c profiles/debug.js & rollup -c profiles/production.js

Unexpected key 'output' found, expected one of: banner, dest, entry, exports, external, footer, format, globals, indent, intro, moduleId, moduleName, onwarn, outro, plugins, sourceMap
Error: Unexpected key 'output' found, expected one of: banner, dest, entry, exports, external, footer, format, globals, indent, intro, moduleId, moduleName, onwarn, outro, plugins, sourceMap
    at validateKeys (C:\.....\esri-leaflet-heatmap\node_modules\rollup\www\ROLLUP\rollup\src\utils\validateKeys.js:12:11)
    at Object.rollup (C:\.....\esri-leaflet-heatmap\node_modules\rollup\www\ROLLUP\rollup\src\rollup.js:39:16)
    at bundle (C:\.....\esri-leaflet-heatmap\node_modules\rollup\bin\runRollup.js:136:16)
    at execute (C:\.....\esri-leaflet-heatmap\node_modules\rollup\bin\runRollup.js:125:3)
    at C:\.....\esri-leaflet-heatmap\node_modules\rollup\bin\runRollup.js:69:4
    at tryCatch (C:\.....\esri-leaflet-heatmap\node_modules\rollup\www\ROLLUP\rollup\node_modules\es6-promise\lib\es6-promise\-internal.js:185:12)
    at invokeCallback (C:\.....\esri-leaflet-heatmap\node_modules\rollup\www\ROLLUP\rollup\node_modules\es6-promise\lib\es6-promise\-internal.js:197:13)
    at publish (C:\.....\esri-leaflet-heatmap\node_modules\rollup\www\ROLLUP\rollup\node_modules\es6-promise\lib\es6-promise\-internal.js:168:7)
    at flush (C:\.....\esri-leaflet-heatmap\node_modules\rollup\www\ROLLUP\rollup\node_modules\es6-promise\lib\es6-promise\asap.js:87:5)
    at process._tickCallback (internal/process/next_tick.js:61:11)
Type rollup --help for help, or visit https://github.com/rollup/rollup/wiki

Per jgravois:

the error is caused by the fact that this plugin imports a base rollup.config from Esri-leaflet, which was upgraded [....] quite awhile ago.

So I think we should update our Rollup dependency in this repo to match esri-leaflet's (v0.56.5 as of this writing) (and any applicable related rollup plugins, etc)

@gavinr
Copy link
Contributor Author

gavinr commented Mar 9, 2022

Fixed in #25

@gavinr gavinr closed this as completed Mar 9, 2022
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

1 participant