graphile / crystal-pre-merge

Repository moved
https://github.com/graphile/crystal
39 stars 10 forks source link

Overhaul peerDeps again #418

Closed benjie closed 1 year ago

benjie commented 1 year ago

Redoing dependencies/peerDependencies around this:

With that being said, Yarn supports "peer dependencies with defaults": if you list a dependency as both a peer and regular dependency, Yarn will use the parent-provided dependency, but will fallback to the regular one if missing. Pnpm supports it too, and I believe even npm supports that somewhat accidentally due to how their "auto-installs" work.

https://github.com/yarnpkg/berry/issues/3710#issuecomment-963732582

On top of that:

Fixes #411 Fixes #412