Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve overal codebase, use modern tech like esbuild and TypeScript 4! #1055

Merged
merged 30 commits into from
Jan 27, 2022

Conversation

RobinMalfait
Copy link
Member

This PR will improve the overal infrastructure of the Headless UI repo.

The biggest change is the removal of TSDX since it is outdated and still on TypeScript version 3. This PR introduces esbuild instead. This also allows us to use newer version of TypeScript.

@vercel
Copy link

vercel bot commented Jan 19, 2022

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

headlessui-vue – ./packages/playground-vue

🔍 Inspect: https://vercel.com/tailwindlabs/headlessui-vue/8QMN5ZNBbe1HMdKcy9FWJ6fze2FE
✅ Preview: https://headlessui-vue-git-improve-codebase-tailwindlabs.vercel.app

headlessui-react – ./packages/playground-react

🔍 Inspect: https://vercel.com/tailwindlabs/headlessui-react/CM5xxJtZQyf2FghkciuGpEoproEv
✅ Preview: https://headlessui-react-git-improve-codebase-tailwindlabs.vercel.app

This will make sure to run a watcher for the actual @headlessui/react
package, and start a development server in the playground-react package.
Still not 100% working, but getting there!
This allows you to run `yarn react playground` or `yarn vue playground`
from the root.
If you minify the code, then it could happen that the errors are a bit
obscure. This will hardcode the component name to improve errors.
@RobinMalfait RobinMalfait changed the title Improve codebase Improve overal codebase, use modern tech like esbuild and TypeScript 4! Jan 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant