search

rocicorp / replicache

Realtime Sync for Any Backend Stack
https://doc.replicache.dev
1.05k stars 37 forks source link

chore: Add test for persist sync head #748

Closed arv closed 2 years ago

arv commented 2 years ago

Test that the sync head gets rewritten during persist.

Also, refactored the static Commit methods. statics are generally a bad idea in JS because tree shaking is having trouble removing dead code. The only valid use case is for "virtual dispatch" of static methods.

vercel[bot] commented 2 years ago

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/rocicorp/replicache/FVUtCtAeJeJkhYcRF6GkJsHgsCQg
✅ Preview: https://replicache-git-fork-arv-test-sync-head-persist-rocicorp.vercel.app

arv commented 2 years ago

I will split this and not be so lazy