search

TriPSs / nx-extend

Nx Workspaces builders and tools
MIT License
164 stars 44 forks source link

Outdated @nx-extend/terraform causes issues with yarn 4 #299

Closed mhp-borisbojic closed 2 months ago

mhp-borisbojic commented 2 months ago

There are two issues with the current version of @nx-extend/terraform

  1. When doing an nx report, it wells me that e.g. @nrwl/workspace should match my installed version of NX, which is odd: I don't have specified @nrwl/workspace in my package.json According to the NX docs, @nrwl/workspace was renamed to @nx/workspace anyway (see also https://github.com/nrwl/nx/issues/27398)
  2. @nrwl/workspace forced outdated versions of nx@npm to be resolved using yarn 4, which results in a failing CI/CD pipeline. Removing your packet removes this issue (see also https://github.com/nrwl/nx/issues/27397)

Is there any update planned on @nx-extend/terraform to match the latest available NX version?

TriPSs commented 2 months ago

Interesting as @nx-extend/terraform does not use @nx/workspace.

mhp-borisbojic commented 2 months ago

Interesting as @nx-extend/terraform does not use @nx/workspace.

But it requests nx@npm which then asks for the other packages:

  - @nrwl/workspace@19.3.0
  - @nrwl/devkit@19.3.0
  - @nrwl/tao@19.3.0
TriPSs commented 2 months ago

Okay, created PR to update Nx to the latest version and adding them as to peerDependencies so it should no longer conflict.