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

fix: Move jsii-spec to Dependencies in jsii-config #1032

Merged

Conversation

MrArnoldPalmer
Copy link
Contributor

@MrArnoldPalmer MrArnoldPalmer commented Nov 19, 2019

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


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Moves jsii-spec to dependencies in package.json instead of devDependencies for jsii-config. This was causing a missing dependency error at runtime.
@MrArnoldPalmer MrArnoldPalmer added bug This issue is a bug. dependencies This issue is a problem in a dependency or a pull request that updates a dependency file. labels Nov 19, 2019
@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@MrArnoldPalmer MrArnoldPalmer merged commit 0581323 into aws:master Nov 19, 2019
@MrArnoldPalmer MrArnoldPalmer deleted the bugfix/jsii-config-spec-dep branch November 19, 2019 21:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug. dependencies This issue is a problem in a dependency or a pull request that updates a dependency file.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

jsii-config is missing a dependency on jsii-spec
3 participants