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

chore: peerDependencies #1091

Merged
merged 1 commit into from
Nov 6, 2018
Merged

chore: peerDependencies #1091

merged 1 commit into from
Nov 6, 2018

Conversation

eladb
Copy link
Contributor

@eladb eladb commented Nov 6, 2018

Adds "peerDependencies" to all modules for all dependencies that
include types that are used in the module's public API.

Also, add "@aws-cdk/cdk" to "peerDependencies" for the typescript
library cdk-init template.

This will be enforced by jsii when aws/jsii#292 is introduced.
Prerequisite for #272
Fixes #979

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

Adds "peerDependencies" to all modules for all dependencies that
include types that are used in the module's public API.

Also, add "@aws-cdk/cdk" to "peerDependencies" for the typescript
library cdk-init template.

This will be enforced by jsii when aws/jsii#292 is introduced.
Prerequisite for #272
Fixes #979
@eladb eladb requested review from RomainMuller and rix0rrr November 6, 2018 13:45
@eladb eladb merged commit 0e3b217 into master Nov 6, 2018
@eladb eladb deleted the benisrae/peer-dependencies branch November 6, 2018 14:18
@NGL321 NGL321 added the contribution/core This is a PR that came from AWS. label Sep 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contribution/core This is a PR that came from AWS.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use of "private" properties prevents multiple versions of a module in the same closure
3 participants