-
Notifications
You must be signed in to change notification settings - Fork 6
feat: React+Tailwind #38
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
Conversation
Really the main issue w/ the code split is we need TOOL_DEFINITIONS, and need to tree shake out all the other junk and avoid e.g. process.env causing an error.... |
Codecov ReportAttention: Patch coverage is
✅ All tests successful. No failed tests found.
Additional details and impacted files@@ Coverage Diff @@
## main #38 +/- ##
===========================================
- Coverage 84.37% 72.46% -11.92%
===========================================
Files 18 22 +4
Lines 2829 2836 +7
Branches 181 188 +7
===========================================
- Hits 2387 2055 -332
- Misses 442 781 +339
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Pushed this into prod and it def works. Will fix tests and merge. (still need to figure out why CF preview envs are totally non existant) |
Swaps our usage to be Vite-based per https://developers.cloudflare.com/workers/vite-plugin/