Uh oh!
There was an error while loading. Please reload this page.
chore: some tweaks to alpha release post - #1830
Conversation
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
Uh oh!
There was an error while loading. Please reload this page.
📝 WalkthroughWalkthroughThis change rewrites the blog post announcing the npmx alpha release. It updates front matter (switching to single-quoted strings and adding Possibly related PRs
🚥 Pre-merge checks | ✅ 1✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (1)
app/pages/blog/alpha-release.md (1)
85-160: Consider adding a brief note about draft-time visibility of federated links.Given
draft: true, a short reviewer-facing note (in PR description or nearby comment) would prevent confusion if some external articles are not yet reachable before coordinated publish.Based on learnings: “webring post URLs added to blog posts may be inaccessible during review if the blog post is still a draft (
draft: true)… coordinated to publish together when the blog post leaves draft mode.”
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
♻️ Duplicate comments (1)
app/pages/blog/alpha-release.md (1)
73-73:⚠️ Potential issue | 🟡 MinorUse the idiomatic preposition: “contribution to GitHub.”
Please change “contribution on GitHub” to “contribution to GitHub” for natural phrasing.
🧹 Nitpick comments (1)
app/pages/blog/alpha-release.md (1)
83-159: Consider adding a short draft-status note for federated links.Since this post is currently a draft, a brief reviewer-facing note (or comment) that some linked community posts may only become accessible once publishing is coordinated would help avoid false alarms during review.
Based on learnings: In app/pages/blog/alpha-release.md for the npmx.dev project, webring post URLs added to blog posts may be inaccessible during review if the blog post is still a draft (
draft: true), and are coordinated to publish together when draft is removed.
🔗 Linked issue
🧭 Context
📚 Description
small tweaks to initial release post. aim is to make it feel less press-release, but I'm not precious - feel free to edit/revert any of these changes 🙏