Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
Lunaria Status Overview🌕 This pull request will trigger status changes. Learn moreBy default, every PR changing files present in the Lunaria configuration's You can change this by adding one of the keywords present in the Tracked Files
Warnings reference
|
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
✅ Files skipped from review due to trivial changes (2)
📝 WalkthroughWalkthroughA package quality scoring feature was added: a new Vue component renders a collapsible quality score section with a circular SVG progress ring and per-check listings. A composable computes scores across five categories (documentation, maintenance, types, bestPractices, security) from package metadata, README, analysis, vulnerability counts/status and provenance. The component is integrated into the package page (main content and sidebar). English and German translations and JSON schema entries were added, and unit and a11y tests were included. Suggested labels
Suggested reviewers
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 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 |
|
thanks for taking a look at this, but i think we really need to spend some time discussing what the right metrics are, how we want to score things (if we want to "score" them at all). i see there's an open issue but we still haven't decided any of this. e.g. do we want a score? do we want just a list of health checks? what should be weighted in what way? etc etc. i think we need to move back to the issue and have what'll probably be a lengthy discussion before we can change any code. |
🔗 Linked issue
Resolves: #48
🧭 Context
With this npmx has a quality and health check inside the detail sidebar.
📚 Description
This checks can looks like:
Maximum possible points are 14.
Tests
Unit tests for point calculations