Conversation
Documents Bun as a first-class runtime (runtime: bun) alongside Node.js, with incompatibility warnings for function serialization, dynamic providers, and magic Lambda functions across relevant pages. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Marked |
Docs Review — PR #17935Scope: 6 files changed (+40 −4) — Bun runtime documentation across docs pages. Overall this is a well-structured PR that adds clear, consistent warnings about Bun's limitations across the relevant pages. A few items to address: Issues1. Shortcode syntax inconsistency (STYLE-GUIDE.md violation) The
These should all use 2. Consider linking to Bun on first mention In Similarly, the intro paragraph at line 28 mentions "Bun runtimes" — that could be the first-mention link instead. Looks good
Mention me ( 🤖 Generated with Claude Code |
|
Your site preview for commit f523855 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-17935-f5238552.s3-website.us-west-2.amazonaws.com. |
|
@claude Fix the issues you found |
|
Claude finished @jkodroff's task in 57s —— View job Fixes applied
All three issues from the previous review have been fixed in commit |
- Use {{% notes %}} instead of {{< notes >}} in lambda.md and function-serialization.md
- Link Bun on first mention in javascript/_index.md
Co-authored-by: Josh Kodroff <jkodroff@users.noreply.github.com>
|
Assigning to @julienp with the intent that he can merge it once this release goes out (assuming the info is correct, and LGTM). |
Documents Bun as a first-class runtime (runtime: bun) alongside Node.js, with incompatibility warnings for function serialization, dynamic providers, and magic Lambda functions across relevant pages.
Fixes pulumi/pulumi#22040
Fixes #17923