fix(skills): rename stale upstream skills and drop articulate - #182
Conversation
Bump PaulRBerg and marketingskills entries to current skill names so fresh CI installs succeed, and remove the empty articulate install-all source. Also fix the awk comment matcher warning in skills-lock. Co-authored-by: Cursor <cursoragent@cursor.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe change updates skill selections and lock metadata, renames two PaulRBerg skills, removes one repository URL, and simplifies the Makefile’s comment-matching expression. ChangesSkills catalog and lock workflow
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Poem
🚥 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 |
Summary
bump-deps/bump-releasetonode-deps-bumper/release-bumperanalytics-trackingtoanalyticshttps://index.how/to/articulateinstall-all source that cannot install skillsskills-lockTest plan
make skills-installinstalls all locked skills (0 not yet installed)Made with Cursor
Summary by cubic
Rename stale skill names to match upstream and drop a broken source so fresh installs and CI pass.
PaulRBerg/agent-skills:bump-deps→node-deps-bumper,bump-release→release-bumper.coreyhaines31/marketingskills:analytics-tracking→analytics.https://index.how/to/articulate.skills-lockMakefile target.Written for commit 50a839a. Summary will update on new commits.