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

Added info for using yarn berry with yarn 3.2.1 #5169

Merged
merged 7 commits into from
Jun 22, 2022

Conversation

Hecatron
Copy link
Contributor

@Hecatron Hecatron commented Jun 6, 2022

Hi,
I had a quick look around at seeing if I could use yarn berry with enableGlobalCache set to true.
I think yarn berry is generally discouraged although I've managed to I think get it working ok with a small change to the svelte.config.js

I suspect there's been some changes to yarn version 3 which now allows it to work (or at least allows the demo app to work)
This bug doesn't seem to exist anymore #2393

Also spotted these related to berry

Currently I'm using Windows and yarn version 3.2.1

@changeset-bot
Copy link

changeset-bot bot commented Jun 6, 2022

⚠️ No Changeset found

Latest commit: 57fba1c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link
Member

@benmccann benmccann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know how much we want to encourage Yarn usage, but I guess if it basically works out-of-the-box then it doesn't hurt to give a suggestion. I'm curious what the other maintainers think though

documentation/faq/80-integrations.md Outdated Show resolved Hide resolved
documentation/faq/80-integrations.md Outdated Show resolved Hide resolved
documentation/faq/80-integrations.md Outdated Show resolved Hide resolved
documentation/faq/80-integrations.md Outdated Show resolved Hide resolved
documentation/faq/80-integrations.md Outdated Show resolved Hide resolved
Copy link
Member

@benmccann benmccann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, but I will leave for the other maintainers to weigh in

@benmccann benmccann added the documentation Improvements or additions to documentation label Jun 7, 2022
@Rich-Harris
Copy link
Member

I think this strikes the right balance between providing people with the right information and steering people towards tools with fewer... idiosyncracies — thanks 👍

@Rich-Harris Rich-Harris merged commit 41baa6f into sveltejs:master Jun 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants