-
Notifications
You must be signed in to change notification settings - Fork 56
Upgrade to Yarn v3, Nx 13.1.4, and move to @nrwl/nx-plugin proper structure #91
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
base: master
Are you sure you want to change the base?
Upgrade to Yarn v3, Nx 13.1.4, and move to @nrwl/nx-plugin proper structure #91
Conversation
09e2e97
to
88c4ab4
Compare
@manfredsteyer @peterbsmyth this is ready to go. |
package.json
Outdated
"test": "nx test", | ||
"lint": "nx workspace-lint && nx lint", | ||
"e2e": "nx e2e", | ||
"affected": "nx affected", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes to package.json
in this commit would be easier to review if you include a commit that sorts the scripts
entries by key name. Essentially you can apply the lint rules like this https://ota-meshi.github.io/eslint-plugin-jsonc/rules/sort-keys.html in a separate commit.
@benpsnyder @PetarKirov this PR is important. when it will merge and release? |
@manfredsteyer I would be willing to maintain and release consistently if you authorize. |
There were issues with running normal CLI upgrade commands, so I had to use the following:
npx @nrwl/tao@latest migrate @nrwl/[email protected] @nrwl/[email protected] --from="@nrwl/[email protected]"
Upgrading to Yarn v3 based on lessons learned from:
Reorganized files based on:
npx create-nx-plugin @angular-architects --pluginName ddd