Closed feugy closed 5 months ago
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
analytics-nextjs | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Feb 20, 2024 1:57pm |
analytics-remix | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Feb 20, 2024 1:57pm |
🌤️ What's in there?
An attempt to reproduce and fix the invalid export issue with Jest.
🧪 How to test?
cd apps/next
pnpm i
pnpm test
To reproduce the issue:
pnpm --ignore-workspace install -D ts-node @vercel/analytics@1.2.1
Then tests will fail:
❗ Note to reviewer
The sole real change in this PR is in
package.json
, were we switch the Node.js export from ESM to CJS file. It's a bit unfair, since the docs says that we could either use CJS or ESM. I guess it's a nasty combination of jest, ts-node and node.