m-ld / m-ld-js

m-ld Javascript engine
https://js.m-ld.org
MIT License
37 stars 2 forks source link

Upgrade `quadstore` and `quadstore-comunica` #154

Closed Peeja closed 1 year ago

Peeja commented 1 year ago

quadstore-communica@3.1.0 requires quadstore@12. We could stick with quadstore-communica@3.0.x, but we may as well keep up with the latest.

Note that quadstore@12 is a breaking change for existing LevelDB data (but not for existing code).

Also, while we're upgrading things, deal with a few security updates from npm audit.

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
m-ld-js ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 20, 2023 5:08pm
Peeja commented 1 year ago

@gsvarovsky This will need a version bump.

gsvarovsky commented 1 year ago

Great! WRT audit problems: before pushing an edge package, I usually do an npm update and use npm outdated to check over major versions I might want to take on. That usually sorts out the audit issues. We can do it here, or wait until text-crdt is merged.

Looking at the situation right now, the things I know can't get upgraded are:

Things that might cause breaking changes (with guess as to ease of fix):