YousefED / SyncedStore

SyncedStore CRDT is an easy-to-use library for building live, collaborative applications that sync automatically.
https://syncedstore.org
MIT License
1.74k stars 53 forks source link

Required version of node/npm for examples #26

Closed jasonm closed 2 years ago

jasonm commented 2 years ago

Thanks for sharing SyncedStore! I'm digging in and starting by playing with the examples.

It looks like the package.json for the todo-react example works with npm 6.14.13 but not npm 8.1.4 (latest stable). Not sure exactly what version started the issue but the message is:

npm ERR! code EWORKSPACESCONFIG
npm ERR! workspaces config expects an Array

Gist of a shell session trying it out: https://gist.github.com/jasonm/33e2d7a7501d3c4f6a8f7df150ed34ec

This seems to fix it:

diff --git a/examples/todo-react/package.json b/examples/todo-react/package.json
index 3ac21ec..4d3cece 100644
--- a/examples/todo-react/package.json
+++ b/examples/todo-react/package.json
@@ -44,10 +44,5 @@
       "last 1 firefox version",
       "last 1 safari version"
     ]
-  },
-  "workspaces": {
-    "nohoist": [
-      "**"
-    ]
   }
 }
YousefED commented 2 years ago

Thanks for reporting! Should be fixed