-
-
Notifications
You must be signed in to change notification settings - Fork 212
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
Compatibility with yarn v2 #238
Comments
I haven't tried |
@arcanis is this a common bug that is solvable easily ? |
What is the exact problem with |
@SASUKE40 This build error happens when I use rollup to bundle a module that I made, and which uses But the same problem happens whenever I import a css file from a third-party module installed with yarn v2. The problem is that yarn v2 stores installed npm module in zip files, not directly in node_modules, and The workaround is to |
Something I find curious is that the error message doesn't make much sense:
It says that a read operation was done on Do you know what's the exact place that's throwing this error? And/or a small repro? |
Here is a repro with another package than |
I explored a bit this repo and also The bug starts here:
Here, as there is no configPath, id is used to set the This path is then passed to This is where the error happens, I didn't go deeper then here. I am not an expert in yarn nor in cosmiconfig so I wouldn't be able to tell you who is wrong here. If you don't need a postcss config, a workaround is:
If you still need one:
|
Funnily the responsibility for this bug is shared by both Yarn and cosmiconfig, as you can still reproduce it by creating a folder named "package.json". But who does that? 😛 Anyway, I've fixed the Yarn part of the problem: yarnpkg/berry#1204 🙂 |
Nice, thanks a lot @arcanis for fixing this 🙏 I believe now I tested with a |
When running in the context of Yarn v2 Pnp, this plugin fails with the error.
Example with
react-grid-layout
:The only workaround is to
yarn unplug react-grid-layout
The text was updated successfully, but these errors were encountered: