Skip to content

chore: remove continuous-learning skill#61

Merged
shunkakinoki merged 2 commits intomainfrom
chore/remove-continuous-learning-skill
Jan 20, 2026
Merged

chore: remove continuous-learning skill#61
shunkakinoki merged 2 commits intomainfrom
chore/remove-continuous-learning-skill

Conversation

@shunkakinoki
Copy link
Owner

@shunkakinoki shunkakinoki commented Jan 20, 2026

Changes

  • Remove continuous-learning skill from .ruler/skills

Testing

  • N/A

Generated with opencode by Claude


Summary by cubic

Removed the continuous-learning skill to deprecate it and eliminate duplicate docs; SKILL.md was deleted from .ruler/skills/continuous-learning and skills/continuous-learning. No runtime impact.

Written for commit 36440e7. Summary will update on new commits.

Copilot AI review requested due to automatic review settings January 20, 2026 23:47
@shunkakinoki shunkakinoki enabled auto-merge (squash) January 20, 2026 23:47
@coderabbitai
Copy link

coderabbitai bot commented Jan 20, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Summary by CodeRabbit

  • Documentation
    • Removed skill documentation and instructional templates related to continuous learning workflows.

✏️ Tip: You can customize this high-level summary in your review settings.

Walkthrough

Two documentation files for a continuous-learning skill template are removed from the repository: one from .ruler/skills/continuous-learning/ and one from skills/continuous-learning/, totaling 250 deleted lines. No new files were added.

Changes

Cohort / File(s) Summary
Continuous-Learning Skill Documentation Removal
\.ruler/skills/continuous-learning/SKILL.md, skills/continuous-learning/SKILL.md
Deleted skill metadata, instructional content, workflow guidelines, extraction criteria, and usage examples (125 lines each).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • #51 — Adds skills/continuous-learning/SKILL.md back (opposite direction of this deletion)
  • #57 — Adds .ruler/skills/continuous-learning/SKILL.md and related setup (opposite direction of this deletion)

Poem

🐰 Files flutter away like autumn leaves,
Two SKILLs vanish—skill deletion believes,
In the space where they dwelled, now just air,
The continuous learner learns not to care! 📚✨

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/remove-continuous-learning-skill

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@shunkakinoki shunkakinoki merged commit 98f176b into main Jan 20, 2026
7 of 8 checks passed
@shunkakinoki shunkakinoki deleted the chore/remove-continuous-learning-skill branch January 20, 2026 23:47
Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes the continuous-learning skill from the repository, which was a meta-skill for extracting and persisting debugging knowledge into the skills repository.

Changes:

  • Deleted the continuous-learning skill documentation from both skills/ and .ruler/skills/ directories

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
skills/continuous-learning/SKILL.md Removed the continuous-learning skill definition and documentation
.ruler/skills/continuous-learning/SKILL.md Removed duplicate continuous-learning skill definition from .ruler directory

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant