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
Yarn2: Use workspace:*
versions
#15413
Conversation
workspace:*
versionsworkspace:*
versions
☁️ Nx Cloud ReportCI is running/has finished running commands for commit 3fc8bdd. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this branch
Sent with 💌 from NxCloud. |
One thing to keep an eye out after this change is that several However, there is an in-progress PR adding this: lerna/lerna#2450 |
@hydrosquall You're 💯 true and some of the changes in this PR are replacements of Lerna commands by Yarn 2 ones or simplification in the scripts. I still need to figure out how to make our Also, we have a strange behavior running |
Fixed in yarnpkg/berry#2255 /
|
Thanks @merceyz! I was staying on Yarn 2.4.2 because of some issues with |
6ee1036
to
235adae
Compare
package.json
Outdated
"dev:tsc": "lerna exec --scope \"@storybook/*\" --parallel -- cross-env-shell node \\$LERNA_ROOT_PATH/scripts/utils/watch-tsc.js", | ||
"dev": "echo \"This command is deprecated and will be removed soon, please use `yarn build` instead\"", | ||
"dev:babel": "echo \"This command is deprecated and will be removed soon, please use `yarn build` instead\"", | ||
"dev:tsc": "echo \"This command is deprecated and will be removed soon, please use `yarn build` instead\"", | ||
"github-release": "github-release-from-changelog", | ||
"lint": "yarn lint:js . && yarn lint:md .", | ||
"lint:js": "cross-env NODE_ENV=production eslint --cache --cache-location=.cache/eslint --ext .js,.jsx,.json,.html,.ts,.tsx,.mjs --report-unused-disable-directives", |
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.
"lint:js": "cross-env NODE_ENV=production eslint --cache --cache-location=.cache/eslint --ext .js,.jsx,.json,.html,.ts,.tsx,.mjs --report-unused-disable-directives", | |
"lint:js": "NODE_ENV=production eslint --cache --cache-location=.cache/eslint --ext .js,.jsx,.json,.html,.ts,.tsx,.mjs --report-unused-disable-directives", |
You can remove cross-env
since Yarn has its own portable shell
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.
Neat! I will take a look at all cross-env
usages across the monorepo
544a16a
to
b39e844
Compare
@shilman I think we will be able to give one more try to the updated release process 😄 |
@gaetanmaisse my forehead's almost healed from the last time we tried to do anything together 🤕 so just let me know whenever is good for you 😂 |
…r workspaces To check the constraint just run `yarn constraints`
b39e844
to
a965266
Compare
Quick note: we now support |
Hi everyone! Seems like there hasn't been much going on in this issue lately. If there are still questions, comments, or bugs, please feel free to continue the discussion. Unfortunately, we don't have time to get to every issue. We are always open to contributions so please send us a pull request if you would like to help. Inactive issues will be closed after 30 days. Thanks! |
# Conflicts: # .yarn/releases/yarn-sources.cjs # .yarnrc.yml # addons/a11y/package.json # addons/actions/package.json # addons/backgrounds/package.json # addons/controls/package.json # addons/docs/package.json # addons/essentials/package.json # addons/jest/package.json # addons/links/package.json # addons/measure/package.json # addons/outline/package.json # addons/storyshots/storyshots-core/package.json # addons/storyshots/storyshots-puppeteer/package.json # addons/storysource/package.json # addons/toolbars/package.json # addons/viewport/package.json # app/angular/package.json # app/ember/package.json # app/html/package.json # app/preact/package.json # app/react/package.json # app/server/package.json # app/svelte/package.json # app/vue/package.json # app/vue3/package.json # app/web-components/package.json # docs/versions/next.json # examples/angular-cli/package.json # examples/cra-kitchen-sink/package.json # examples/cra-react15/package.json # examples/cra-ts-essentials/package.json # examples/cra-ts-kitchen-sink/package.json # examples/ember-cli/package.json # examples/html-kitchen-sink/package.json # examples/official-storybook/package.json # examples/preact-kitchen-sink/package.json # examples/react-ts-webpack4/package.json # examples/react-ts/package.json # examples/server-kitchen-sink/package.json # examples/standalone-preview/package.json # examples/svelte-kitchen-sink/package.json # examples/vue-3-cli/package.json # examples/vue-cli/package.json # examples/vue-kitchen-sink/package.json # examples/web-components-kitchen-sink/yarn.lock # lerna.json # lib/addons/package.json # lib/api/package.json # lib/builder-webpack4/package.json # lib/builder-webpack5/package.json # lib/channel-postmessage/package.json # lib/channel-websocket/package.json # lib/cli-sb/package.json # lib/cli-storybook/package.json # lib/cli/package.json # lib/client-api/package.json # lib/codemod/package.json # lib/components/package.json # lib/core-client/package.json # lib/core-common/package.json # lib/core-server/package.json # lib/core/package.json # lib/manager-webpack4/package.json # lib/manager-webpack5/package.json # lib/router/package.json # lib/source-loader/package.json # lib/theming/package.json # lib/ui/package.json # package.json # scripts/build-package.js # scripts/run-registry.ts # yarn.lock
@shilman @gaetanmaisse @merceyz I did my best to get this updated with What the course of action here? Are we proceeding? Are we taking this and redo-ing it on Let's make this PR a draft, and discuss if it's salvageable? |
Also ping-ing @arcanis And @hydrosquall ☝️ Can you tell me what the roadmap is for this PR? |
I also tried to update it a few weeks ago, a nightmare and at the end it wasn't working anymore.
I've in mind to try to redo it on |
@shilman and everyone else, you could finish this PR by switching to Lerna-Lite which did add support to |
What I did
workspace:*
version, for details see: https://yarnpkg.com/features/workspaces#workspace-ranges-workspacebuild
NPM script--
Commands for the release: