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

Build is broken due to external dependency hell #160

Closed
dmcnamara-eng opened this issue Jul 12, 2022 · 10 comments
Closed

Build is broken due to external dependency hell #160

dmcnamara-eng opened this issue Jul 12, 2022 · 10 comments

Comments

@dmcnamara-eng
Copy link
Collaborator

ERR_PNPM_PEER_DEP_ISSUES  Unmet peer dependencies

ember-headlessui
└─┬ release-it
  └─┬ @octokit/rest
    └─┬ @octokit/plugin-paginate-rest
      └── ✕ unmet peer @octokit/core@>=4: found 3.6.0 in @octokit/rest
@NullVoxPopuli
Copy link
Collaborator

NullVoxPopuli commented Jul 12, 2022

yeah, octokit (or release-it) needs to fix that -- nothing we can really do here

@dmcnamara-eng
Copy link
Collaborator Author

Indeed. I guess no harm here unless someone is using ember-headlessui@latest

yeah, octokit (or release-it) needs to fix that -- nothing we can really do here

We cannot release new version of ember-headlessui until that is fixed.

@NullVoxPopuli
Copy link
Collaborator

We cannot release new version of ember-headlessui until that is fixed.

correct.

So why do we have 3.6.0?

❯ pnpm -r  why @octokit/core 
ember-headlessui@0.12.3 /home/nullvoxpopuli/Development/NullVoxPopuli/ember-headlessui/ember-headlessui

devDependencies:
release-it 14.14.3
└─┬ @octokit/rest 18.12.0
 ├── @octokit/core 3.6.0
 ├─┬ @octokit/plugin-paginate-rest 2.21.1
 │ └── @octokit/core 3.6.0 peer
 ├─┬ @octokit/plugin-request-log 1.0.4
 │ └── @octokit/core 3.6.0 peer
 └─┬ @octokit/plugin-rest-endpoint-methods 5.13.0
   └── @octokit/core 3.6.0 peer
release-it-lerna-changelog 3.1.0
└─┬ release-it 14.14.3 peer
 └─┬ @octokit/rest 18.12.0
   ├── @octokit/core 3.6.0
   ├─┬ @octokit/plugin-paginate-rest 2.21.1
   │ └── @octokit/core 3.6.0 peer
   ├─┬ @octokit/plugin-request-log 1.0.4
   │ └── @octokit/core 3.6.0 peer
   └─┬ @octokit/plugin-rest-endpoint-methods 5.13.0
     └── @octokit/core 3.6.0 peer

latest release-it is 15.1.1, we are not using that -- so maybe the problem is release-it. Let's upgrade that

@NullVoxPopuli
Copy link
Collaborator

I upgraded release-it, but release-it-lerna-changelog has not been updated

ember-headlessui
├─┬ release-it
│ └─┬ @octokit/rest
│   └─┬ @octokit/plugin-paginate-rest
│     └── ✕ unmet peer @octokit/core@>=4: found 3.6.0 in @octokit/rest
└─┬ release-it-lerna-changelog
  └── ✕ unmet peer release-it@^14.0.0: found 15.1.1

@NullVoxPopuli
Copy link
Collaborator

There is already an issue: release-it-plugins/lerna-changelog#203

But rwjblue is on vacation, I think

@NullVoxPopuli
Copy link
Collaborator

ah ok, so release-it is using @octokit/rest v18, instead of v19

@NullVoxPopuli
Copy link
Collaborator

This should resolve : #161

@dmcnamara-eng
Copy link
Collaborator Author

Nice one! I went down that road but wasn't sure on our "policy" on using forks.

@NullVoxPopuli
Copy link
Collaborator

it's a dev-time tool, so it's not going to affect our users 🙈

@NullVoxPopuli
Copy link
Collaborator

resolved by: #163

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 a pull request may close this issue.

2 participants