GavinJoyce / ember-headlessui

https://gavinjoyce.github.io/ember-headlessui/
Other
92 stars 34 forks source link

Build is broken due to external dependency hell #160

Closed dmcnamara-eng closed 2 years ago

dmcnamara-eng commented 2 years ago
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 commented 2 years ago

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

dmcnamara-eng commented 2 years ago

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 commented 2 years ago

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 commented 2 years ago

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 commented 2 years ago

There is already an issue: https://github.com/rwjblue/release-it-lerna-changelog/issues/203

But rwjblue is on vacation, I think

NullVoxPopuli commented 2 years ago

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

NullVoxPopuli commented 2 years ago

This should resolve : https://github.com/GavinJoyce/ember-headlessui/pull/161

dmcnamara-eng commented 2 years ago

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

NullVoxPopuli commented 2 years ago

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

NullVoxPopuli commented 2 years ago

resolved by: https://github.com/GavinJoyce/ember-headlessui/pull/163