-
-
Notifications
You must be signed in to change notification settings - Fork 111
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
Replace @unly/utils-simple-logger and winston by @unly/simple-logger and chalk (reduces bundle size by 40Kb) #314
Replace @unly/utils-simple-logger and winston by @unly/simple-logger and chalk (reduces bundle size by 40Kb) #314
Conversation
❌ Deployment FAILED |
✅ Deployment SUCCESS |
✅ E2E tests SUCCESS for commit 39385f1 previously deployed Storybook static site at https://nrn-v2-mst-aptd-at-lcz-sty-storybook-ki08viij3-unly-oss.vercel.app |
…re, tried tons of things nothing works :/
✅ Deployment SUCCESS |
✅ Deployment SUCCESS |
✅ E2E tests SUCCESS for commit 0dbfa1b previously deployed at https://nrn-v2-mst-aptd-at-lcz-sty-c1-9oboeul1p-unly-oss.vercel.app |
✅ E2E tests SUCCESS for commit 0dbfa1b previously deployed Storybook static site at https://nrn-v2-mst-aptd-at-lcz-sty-storybook-d298w61sm-unly-oss.vercel.app |
✅ Deployment SUCCESS |
✅ Deployment SUCCESS |
✅ E2E tests SUCCESS for commit 0c411b9 previously deployed at https://nrn-v2-mst-aptd-at-lcz-sty-c1-v669ipe0w-unly-oss.vercel.app |
✅ E2E tests SUCCESS for commit 0c411b9 previously deployed Storybook static site at https://nrn-v2-mst-aptd-at-lcz-sty-storybook-72naeykb9-unly-oss.vercel.app |
…and chalk (reduces bundle size by 40Kb) (#314)
This replaces the @unly/utils-simple-logger by a new @unly/simple-logger package.
The
winston
dependency has been removed, and been replaced bychalk
(8kb gzipped) which is only being imported on the server.The new package is <1Kb on the browser.