Skip to content

Update PR skill - No internal links#133

Merged
carole-lavillonniere merged 1 commit intomainfrom
pr-skill
Mar 18, 2026
Merged

Update PR skill - No internal links#133
carole-lavillonniere merged 1 commit intomainfrom
pr-skill

Conversation

@carole-lavillonniere
Copy link
Copy Markdown
Collaborator

@carole-lavillonniere carole-lavillonniere commented Mar 18, 2026

PR description should not include internal links since the repo is open source.

@carole-lavillonniere carole-lavillonniere marked this pull request as ready for review March 18, 2026 10:39
Copy link
Copy Markdown
Member

@silv-io silv-io left a comment

Choose a reason for hiding this comment

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

LGTM!

@carole-lavillonniere carole-lavillonniere merged commit df59674 into main Mar 18, 2026
6 of 7 checks passed
@carole-lavillonniere carole-lavillonniere deleted the pr-skill branch March 18, 2026 10:41
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 18, 2026

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

This change updates the PR creation skill template by removing the Related section from the PR body structure and simplifying Linear ticket references. The Linear ticket fetch now retrieves only title and description instead of including URL, and the PR body now ends with a ticket reference using "Closes" or "Towards" keywords.

Changes

Cohort / File(s) Summary
PR Template Simplification
.claude/skills/create-pr/SKILL.md
Removed Related section from PR body structure; eliminated Linear ticket URL fetching; updated PR body rules to reference tickets with "Closes" or "Towards" keywords instead of linking to Linear.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • anisaoshafi
🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Update PR skill - No internal links' directly relates to the main change: removing internal links from the PR skill guidance.
Description check ✅ Passed The description 'PR description should not include internal links since the repo is open source' is directly related to the changeset which removes internal links from PR templates.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch pr-skill
📝 Coding Plan
  • Generate coding plan for human review comments

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.

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.

2 participants