Skip to content
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

Docs: add warning callout for Yarn Berry #41036

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

julien-deramond
Copy link
Member

@julien-deramond julien-deramond commented Nov 17, 2024

Description

Based on the discussion in #40933 and the solution brought to build our Sass & JS example in #40933 (comment), this PR suggests a small enhancement to at leasts mention this "trick".

The approach is that we consider that most folks are still on Yarn (Classic), but we mention a way to make our examples work with Yarn Berry too to unblock some others.

Another approach would be to create a dedicated example in twbs/examples and link to it. But, TBH, I don't have time for it, and I'm not sure whether it's needed for now.

Type of changes

  • Docs enhancement

Checklist

  • I have read the contributing guidelines
  • My code follows the code style of the project (using npm run lint)
  • My change introduces changes to the documentation
  • I have updated the documentation accordingly
  • (N/A) I have added tests to cover my changes
  • All new and existing tests passed

Live previews

@julien-deramond julien-deramond marked this pull request as ready for review November 17, 2024 18:30
@julien-deramond julien-deramond force-pushed the main-jd-add-warning-callout-for-yarn-berry branch from 62ed3e2 to 5ff09a8 Compare November 17, 2024 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Needs review
Development

Successfully merging this pull request may close these issues.

1 participant