Repository navigation
Update fullstack-dev-server.md to include {bunFullstack: true} - #884
ZeroStride wants to merge 1 commit into
Conversation
Add `bunFullstack: true` since bundling does not occur automatically anymore.
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Enable Bun Fullstack Dev Server docs/patterns/fullstack-dev-server.md |
Both setup examples pass bunFullstack: true to staticPlugin. The custom-prefix example retains prefix: '/'. |
Priority:
Estimated code review effort: 1 (Trivial) | ~5 minutes
Change: Other
Possibly related issues
- Static building fails with "@elysia/static", succeeds with "@elysiajs/static" elysia#2030 — The issue concerns a fullstack
bun devsetup that stopped working after a package change. This PR updates the linked tutorial’s setup examples, though the issue cause is not confirmed.
Suggested reviewers: saltyaom
Merge Risk: 
Minimal · up to 72665
-
The examples consistently enable Bun fullstack bundling, and the custom prefix remains preserved.
Pre-merge checks: 
5
-
Full details: Title check
-
Explanation
Baka~! The title says what changed: the fullstack dev server doc now includes {bunFullstack: true}. It fits the PR. The changed file is docs/patterns/fullstack-dev-server.md, and the PR says bundling no longer happens automatically. The title is clear and specific, so it passes. Ehehe~ ♡ (=^・ω・^=)
Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
-
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Two examples gain a setting, neat and bright~staticPlugin gets its flag just right ♡
The prefix stays, exactly as before~
No extra magic hiding in the lore~
There, now your setup’s less confused~ (¬‿¬)
Comment @coderabbitai help to get the list of available commands.
Add
bunFullstack: truesince bundling does not occur automatically anymore.Summary by CodeRabbit