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/pmndrs/jotai/FRyxyWfEzW433iqM1LFQd4FrPdrB
✅ Preview:https://jotai-git-fork-liby-master-pmndrs.vercel.app

This pull request is automatically built and testable inCodeSandbox.

To see build info of the built libraries, clickhere or the icon next to each commit SHA.

Latest deployment of this branch, based on commit1e36c08:

is this really necessary?

This would be just a preference, but it helps to me to understand dependencies across bundles. So, it might not be really necessary, but make things consistent.

I also confirmed, tsconfig change doesn't affect bundles.

$ git diff tsconfig.jsondiff --git a/tsconfig.json b/tsconfig.jsonindex 9f5dd48..3f3d107 100644--- a/tsconfig.json+++ b/tsconfig.json@@ -2,7 +2,7 @@   "compilerOptions": {     "target": "es2019",     "strict": true,-    "jsx": "preserve",+    "jsx": "react-jsx",     "allowSyntheticDefaultImports": true,     "esModuleInterop": true,     "moduleResolution": "node",$ diff -r dist.jsx_preserve dist.jsx_react-jsx$

This would be just a preference, but it helps to me to understand dependencies across bundles. So, it might not be really necessary, but make things consistent.

I see what you mean.

Thanks for the work. Much appreciated.

That's alright, glad to help!

For now, it looks good to me. (examples may not work with old TS, but let's hope no one complain.)

If there are any problems in the future, let's communicate at that time to see how to modify them.