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

Convert tailwind package to TypeScript #686

Merged
merged 5 commits into from
Feb 19, 2020

Conversation

joestrouth1
Copy link
Contributor

Part of #668

Makes the requested changes to tailwind for conversion to TS, plus strict: true in the copied tsconfig. No anys in the generated types, though there are some unknowns. The tailwind config passed in could be anything.

Does not update the tests. I'm not sure what the plan is for converting to ts-jest or similar.

@joestrouth1 joestrouth1 marked this pull request as ready for review February 17, 2020 22:53
Copy link
Member

@mxstbr mxstbr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Amazing, thank you for digging into this! 💯 Now that #674 was merged we are testing with ts-jest, so could you also change the .test.js to a .test.ts? 🙏

@joestrouth1
Copy link
Contributor Author

Test changed!

@mxstbr mxstbr mentioned this pull request Feb 19, 2020
32 tasks
Copy link
Member

@mxstbr mxstbr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Amazing! 👏 Thanks so much for helping out here @joestrouth1! 🔥

@mxstbr mxstbr merged commit 1b72aef into system-ui:master Feb 19, 2020
@joestrouth1 joestrouth1 deleted the ts-tailwind branch February 19, 2020 22:49
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.

None yet

2 participants