build(deps): bump Lunaria#234
Conversation
Dokploy Preview Deployment
|
📝 WalkthroughWalkthroughUpdates git tracking timestamp references from Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
Tip Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord. 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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@package.json`:
- Line 54: The dependency reference for "@lunariajs/core" uses the ephemeral
pkg.pr.new URL which risks future install breakage; change the package entry for
"@lunariajs/core" to use the official npm semver form (replace the URL value
with "^0.1.1") so package resolution uses the npm registry instead of the
pkg.pr.new build.
ℹ️ Review info
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yaml
📒 Files selected for processing (3)
lunaria/components.tspackage.jsonscripts/lunaria-report-bot.ts
dreyfus92
left a comment
There was a problem hiding this comment.
Thank you @ArmandPhilippot 😄
|
@coderabbitai please unblock your review, the changes here are needed to prevent pkg.pr.new dead links in CI |
|
✅ Actions performedComments resolved and changes approved. |
Description
Bumps Lunaria with a more recent package to avoid similar issues as in Astro Docs because of an old package that suddenly vanish.
Based on withastro/docs#12066 and withastro/docs#13285 these are the only changes required!
Summary by CodeRabbit