Skip to content

Conversation

@AntoLC
Copy link
Collaborator

@AntoLC AntoLC commented Apr 7, 2025

Purpose

We cannot add new js dependency locally when we bind the frontend with the y-provider service. It results in
"EPERM: operation not permitted" when the node_modules has to be updated.
Better to remove the binding, we can add the binding locally during development on the y-provider.

image

@AntoLC AntoLC added bug Something isn't working docker labels Apr 7, 2025
@AntoLC AntoLC requested a review from lunika April 7, 2025 10:55
@AntoLC AntoLC self-assigned this Apr 7, 2025
@AntoLC AntoLC force-pushed the fix/y-provider-binding branch from d37f04d to dfa705f Compare April 7, 2025 10:58
@lunika
Copy link
Member

lunika commented Apr 28, 2025

restarting it does not resolve the issue ?
Are you updating the nodemon dependency when you have this error ?

@thepearl
Copy link

same, i had to comment out the volumes line in docker-compose file in order to run it properly.

@AntoLC
Copy link
Collaborator Author

AntoLC commented May 2, 2025

restarting it does not resolve the issue ? Are you updating the nodemon dependency when you have this error ?

No, I just add dependencies on the app "impress", not even on the "y-provider". But it is a monorepo, so the node_modules is shared.

We cannot add new js dependency locally when we bind the
frontend with the y-provider service. It results in
"EPERM: operation not permitted" when the `node_modules`
has to be updated.
Better to remove the binding, we can add the binding
locally during development on the y-provider.
@AntoLC AntoLC force-pushed the fix/y-provider-binding branch from dfa705f to dfe8ae1 Compare May 6, 2025 08:37
@AntoLC AntoLC merged commit dfe8ae1 into main May 6, 2025
19 of 21 checks passed
@AntoLC AntoLC deleted the fix/y-provider-binding branch May 6, 2025 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working docker

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants