Skip to content

fix(ci): enforce PR checklist completion with evidence links#368

Merged
mcj-coder merged 2 commits intomainfrom
fix/290-pr-checklist-enforcement
Jan 15, 2026
Merged

fix(ci): enforce PR checklist completion with evidence links#368
mcj-coder merged 2 commits intomainfrom
fix/290-pr-checklist-enforcement

Conversation

@mcj-coder
Copy link
Owner

@mcj-coder mcj-coder commented Jan 15, 2026

Summary

  • Change DangerJS Rule 5b from warn to fail for missing evidence links
  • Create PR template (.github/PULL_REQUEST_TEMPLATE.md) with required evidence format
  • Add "PR Evidence Requirements" section to CONTRIBUTING.md with format examples

Closes #290

Test plan

🤖 Generated with Claude Code

- Change DangerJS Rule 5b from warn to fail for missing evidence links
- Create PR template with required evidence format examples
- Add PR Evidence Requirements section to CONTRIBUTING.md
- Reference exemplar PR #263 for proper evidence format

Refs: #290

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@mcj-coder mcj-coder enabled auto-merge (squash) January 15, 2026 14:09
@github-actions
Copy link

github-actions bot commented Jan 15, 2026

Messages
📖 All PR validation checks passed.

Generated by 🚫 dangerJS against b4b4939

Copy link
Collaborator

@martincjarvis martincjarvis left a comment

Choose a reason for hiding this comment

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

QA Approval

Verification

  • DangerJS blocks unchecked items (verified: Rule 5a uses fail)
  • DangerJS fails on missing evidence (verified: Rule 5b changed from warn to fail)
  • PR template created with evidence format examples
  • CONTRIBUTING.md has "PR Evidence Requirements" section
  • All CI checks pass
  • DangerJS enforcement working (this PR itself was blocked until evidence added)

Test of New Rule

The new enforcement rule successfully blocked this PR until proper evidence links were added - demonstrating the rule works as intended.

Approved for merge.

@mcj-coder mcj-coder merged commit 15157ad into main Jan 15, 2026
6 checks passed
@mcj-coder mcj-coder deleted the fix/290-pr-checklist-enforcement branch January 15, 2026 14:39
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.

fix(process): Enforce PR checklist completion with evidence links

2 participants