Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

Make React a peer dependency #268

Closed
wants to merge 1 commit into from
Closed

Conversation

wmertens
Copy link
Collaborator

@wmertens wmertens commented Jun 2, 2016

This project fails if the React is not v15+, and you can't have your own version of React because webpack will bundle the one from the project that uses Carte Blanche.

This project fails if the React is not v15+, and you can't have your own version of React because webpack will bundle the one from the project that uses Carte Blanche.
@nikgraf
Copy link
Member

nikgraf commented Jun 26, 2016

I will close this as peerDepedencies are npm2 and we want to move forward. Let's define engine npm3 and only focus on this. see https://codingwithspike.wordpress.com/2016/01/21/dealing-with-the-deprecation-of-peerdependencies-in-npm-3/

@nikgraf nikgraf closed this Jun 26, 2016
@wmertens
Copy link
Collaborator Author

Err, that doesn't actually solve the problem, you still need to declare React 15+ as your peer dependency because Carte Blanche doesn't work with 0.14 and there can only be one React in the project, so the project needs to use React 15+ too…

@nikgraf
Copy link
Member

nikgraf commented Jun 26, 2016

How would the peer dependency help here especially with npm3?

@wmertens
Copy link
Collaborator Author

It would tell the dev that is installing carte-blanche if their version of
React is wrong. They would get an invalid peer dep error.

On Mon, Jun 27, 2016 at 12:11 AM Nik Graf notifications@github.com wrote:

How would the peer dependency help here especially with npm3?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#268 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AADWlvdNXlLDIgQ39HypYGHEhIcATYRLks5qPvkEgaJpZM4IsrwC
.

Wout.
(typed on mobile, excuse terseness)

@nikgraf
Copy link
Member

nikgraf commented Jul 1, 2016

@wmertens I'm pretty sure in npm3 you get an invalid dep error as it flattens out them. Is this not correct?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants