mono - chore: upgrading typescript, tsdown, and docula#183
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #183 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 10 10
Lines 1266 1266
Branches 232 231 -1
=========================================
Hits 1266 1266 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Code Review
This pull request updates several dependencies in package.json and pnpm-lock.yaml, including upgrades to docula, tsdown, typescript, and various @ai-sdk packages. Additionally, the website:serve script in package.json was renamed to website:dev. I have provided a suggestion to include a website:serve script that runs docula serve to facilitate production previewing, which complements the existing development workflow.
| "test:services:stop": "docker compose down", | ||
| "website:build": "docula build", | ||
| "website:serve": "docula dev", | ||
| "website:dev": "docula dev", |
There was a problem hiding this comment.
Renaming website:serve to website:dev correctly aligns the script name with the docula dev command. However, it is common practice to also have a serve script for previewing the production build (usually after running website:build). Consider adding a website:serve script that executes docula serve to provide a complete development and preview workflow.
| "website:dev": "docula dev", | |
| "website:dev": "docula dev", | |
| "website:serve": "docula serve", |
Please check if the PR fulfills these requirements
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
mono - chore: upgrading typescript, tsdown, and docula