Skip to content

chore: add compatibility for Nx 16#3589

Merged
escapedcat merged 1 commit intoconventional-changelog:masterfrom
fguitton:chore/nx-16
May 7, 2023
Merged

chore: add compatibility for Nx 16#3589
escapedcat merged 1 commit intoconventional-changelog:masterfrom
fguitton:chore/nx-16

Conversation

@fguitton
Copy link
Copy Markdown
Contributor

Description

This is still a work in progress until Nx 16 release (anytime now)
This PR adjusts the package.json Nx compatibility for commitlint and move tests for Nx 14 and Nx 15 separately. This is due to changes in the packages name for Nx from @nrwl/* to @nx/*

Motivation and Context

Nx is about to push version 16.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@fguitton fguitton force-pushed the chore/nx-16 branch 4 times, most recently from 318ccd0 to 03d2ab1 Compare April 23, 2023 21:35
@escapedcat
Copy link
Copy Markdown
Member

Thanks for taking care of this!

Comment thread @commitlint/config-nx-scopes/fixtures/empty/package.json Outdated
@fguitton fguitton marked this pull request as ready for review May 6, 2023 14:32
@fguitton
Copy link
Copy Markdown
Contributor Author

fguitton commented May 7, 2023

@escapedcat This should be ready now 😝

While Nx 16 provides virtually full compatibility with Nx 15 it has deprecated numerous internal structures and commitlint will need to move away from using the Workspace provider to extract scopes. I will try to get a PR out when I can find some time.

There is also perhaps the topic of Lerna, which is basically just Nx under the hood nowadays. Perhaps some unifying would be in order ?

@escapedcat escapedcat merged commit b335b83 into conventional-changelog:master May 7, 2023
@escapedcat
Copy link
Copy Markdown
Member

Thanks @fguitton !

There is also perhaps the topic of Lerna, which is basically just Nx under the hood nowadays. Perhaps some unifying would be in order ?

Happy for any suggestions. The lerna package still seems to be quite popular.

@fguitton fguitton deleted the chore/nx-16 branch May 9, 2023 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants