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

Remove @oclif/core dependency / change to peer dependency #471

Open
dsokal opened this issue Dec 22, 2021 · 1 comment
Open

Remove @oclif/core dependency / change to peer dependency #471

dsokal opened this issue Dec 22, 2021 · 1 comment

Comments

@dsokal
Copy link

dsokal commented Dec 22, 2021

Currently, @oclif/core depends on cli-ux which depends on @oclif/core, and which depends on cli-ux, and so on.
This cycle can cause (and actually causes) multiple versions of @oclif/core and cli-ux exist in node_modules.

There are a few solutions to this problem, e.g.:

  • You could add @oclif/core as a peer dependency.
  • You could extract the common functionality to a separate package.
@RodEsp
Copy link
Contributor

RodEsp commented Dec 22, 2021

Removing circular dependencies is certainly something we've been working on recently. Have done this for a few other repos already and will eventually get to these at some point too. Thanks for pointing it out!

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

No branches or pull requests

2 participants