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.

Sign up for GitHub

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

Jump to bottom

jsii-config is missing a dependency on jsii-spec #1030

Closed
eladb opened this issue Nov 19, 2019 · 0 comments · Fixed by #1032
Closed

jsii-config is missing a dependency on jsii-spec #1030

eladb opened this issue Nov 19, 2019 · 0 comments · Fixed by #1032
Assignees
@MrArnoldPalmer
Labels
bug This issue is a bug. p0

Comments

@eladb
Copy link
Contributor

eladb commented Nov 19, 2019

🐛 Bug Report

❯ npx jsii-config
npx: installed 65 in 9s
Cannot find module 'jsii-spec

General Information

  • JSII Version: 0.20.7.
@eladb eladb added the bug This issue is a bug. label Nov 19, 2019
@eladb eladb added the p0 label Nov 19, 2019
@eladb eladb mentioned this issue Nov 19, 2019
Closed
@MrArnoldPalmer MrArnoldPalmer mentioned this issue Nov 19, 2019
Merged
MrArnoldPalmer added a commit that referenced this issue Nov 19, 2019
@MrArnoldPalmer
fix: Move jsii-spec to Dependencies (#1032)
0581323 
Moves jsii-spec to dependencies in package.json instead of devDependencies for jsii-config. This was causing a missing dependency error at runtime.

Fixes #1030
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MrArnoldPalmer MrArnoldPalmer

Labels
bug This issue is a bug. p0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: Move jsii-spec to Dependencies in jsii-config MrArnoldPalmer/jsii
2 participants
@eladb @MrArnoldPalmer