Skip to content

The reflect-metadata package should be a peer dependency.#2

Open
jloveridge wants to merge 1 commit intobitrinjani:masterfrom
gradecam:feature/peer-reflect
Open

The reflect-metadata package should be a peer dependency.#2
jloveridge wants to merge 1 commit intobitrinjani:masterfrom
gradecam:feature/peer-reflect

Conversation

@jloveridge
Copy link

I have encountered issues when multiple dependencies of a project depend on the reflect-metadata package. When this occurs I have seen tsc fail to transpile the project source. This has, in my experience, been avoidable by listing reflect-metadata in peerDependencies instead of dependencies and allowing the project itself to install one version of the reflect-metadata package.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 7c4dbbf on gradecam:feature/peer-reflect into 109f38a on bitrinjani:master.

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

Successfully merging this pull request may close these issues.

2 participants