Skip to content

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

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

benpsnyder
Copy link

@benpsnyder benpsnyder commented Dec 1, 2021

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

@benpsnyder benpsnyder force-pushed the task/upgrade-to-yarn-v3 branch from 09e2e97 to 88c4ab4 Compare December 1, 2021 21:00
@benpsnyder benpsnyder changed the title Upgrade to Yarn v3 and Nx 13.1.4 Upgrade to Yarn v3, Nx 13.1.4, and move to @nrwl/nx-plugin proper structure Dec 1, 2021
@benpsnyder
Copy link
Author

@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",

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.

@jon9090
Copy link

jon9090 commented Dec 17, 2021

@benpsnyder @PetarKirov this PR is important. when it will merge and release?

@benpsnyder
Copy link
Author

@manfredsteyer I would be willing to maintain and release consistently if you authorize.

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 this pull request may close these issues.

3 participants