fix: remove duplicate ssr-lit module entry - #1601
lonewolfyx wants to merge 1 commit into
Conversation
Both modules/ssr-lit.yml and modules/nuxt-ssr-lit.yml describe the same repository (prashantpalikhe/nuxt-ssr-lit) and declare the same module name `ssr-lit`. Keep nuxt-ssr-lit.yml, which already contains all of the information from ssr-lit.yml plus the current Nuxt 3 metadata, and drop the outdated Nuxt 2 entry.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review. 📝 WalkthroughWalkthroughThe Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This change removes a duplicate module registry entry. No current merge-readiness risk is identified. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
The modules directory currently contains two entries that point to the same repository (
prashantpalikhe/nuxt-ssr-lit) and declare the same module namessr-lit:modules/ssr-lit.yml— the legacy Nuxt 2 entry (nuxt: ^2.0.0, categoryDevtools, one maintainer, no description)modules/nuxt-ssr-lit.yml— the current entry (nuxt: '>=3.0.0', categoryLibraries, two maintainers, description)The two entries collide on
name, so only one of them can be resolved correctly.Since
nuxt-ssr-lit.ymlalready contains every piece of information fromssr-lit.yml(repo, npm, github, website, maintainer) plus the more complete and current metadata, this PR removes the outdatedssr-lit.yml. No other files reference it.