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

Fix run_sad #1

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

Conversation

markvandenbrink
Copy link

@markvandenbrink markvandenbrink commented Jun 9, 2021

This problem is related to this issue: yarnpkg/yarn#685

The problem is that yarn copies the entire folder from ./tripetto-block-text/ to ./app/node_modules/tripetto-block-text. This also includes the node_modules folder in ./tripetto-block-text/node_modules and this folder also contains a version of the tripetto package. So now, the text block will use this local instance of Tripetto to register itself to instead of the one used by your app.

The solution here is to run npm pack which respects the .npmignore file. This will exclude some folders including node_modules. After that, the resulting tarball can be used to install the package.

This problem is related to this issue: yarnpkg/yarn#685

The problem is that yarn copies the entire folder from `./tripetto-block-text/` to `./app/node_modules/tripetto-block-text`. This also includes the `node_modules` folder in `./tripetto-block-text/node_modules`. This folder also contains the `tripetto` package. So now, the text block will use this locale instance of Tripetto to register the block instead of the one used by the app.
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.

1 participant