Skip to content

test(pr-kit): prove the merge-danger scorer runs - #214

Closed
AojdevStudio wants to merge 2 commits into
mainfrom
test/scorer-proof
Closed

AojdevStudio wants to merge 2 commits into
mainfrom
test/scorer-proof

Conversation

@AojdevStudio

@AojdevStudio AojdevStudio commented Nov 8, 2026 •

Copy link
Copy Markdown
Owner

Problem

PR #213 installed the merge-danger scorer, and no materialize-3d PR is open to show it runs. The new homelab-ci runner has not yet posted a merge-danger rating.

Change

An empty commit, then a one-line scratch file, so the scorer has a diff to rate. This PR closes without merging.

1089e4c test(pr-kit): prove the merge-danger scorer runs       (empty)
5fe8207 test(pr-kit): give the scorer a one-line diff to rate  (+ scorer-proof.txt)

Proof

Check Before After
merge-danger on 1089e4c, empty diff n/a error: "Not rated: the diff is empty, so there is nothing to rate." from run 42480486187 on github-runner-ci-materialize-3d
git diff --name-only origin/main HEAD on 5fe8207 n/a scorer-proof.txt only

Review

No findings.

Merge danger

Door: two-way. It closes unmerged.
Blast radius: none. It adds one scratch text file that nothing reads.

Changes made by Claude Opus 5.5 via Claude Code

Empty commit. This PR exists only to trigger merge-danger.yml once and is
closed without merging.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coldtea-pr-lens

Copy link
Copy Markdown

Note

The title starts with test(pr-kit):, so PR Lens left this pull request undrawn. Comment @pr-lens draw to draw it

github.comment.notice: false in .github/pr-lens.yml turns this note off

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Nov 8, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 48 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 713a407e-53c8-44d2-864b-d274c4ba6c15
Commits

Reviewing files that changed from the base of the PR and between 1089e4c and 5fe8207.

Files selected for processing: 1

  • scorer-proof.txt
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@pullfrog pullfrog Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ No new issues found.

Reviewed changes The reviewed commits add a one-line scratch file to provide a non-empty diff for exercising the merge-danger scorer; the earlier commit is empty.

  • Scorer test stimulus scorer-proof.txt contains only a note explaining its purpose, and the PR states it will close without merging.

Pullfrog  | View workflow run | Using gpt-6-luna | 𝕏

@AojdevStudio

Copy link
Copy Markdown
Owner Author

Proof done. merge-danger.yml run 42480541160 finished on github-runner-ci-materialize-3d and posted success on 5fe8207: Blast radius none. danger.ts verified it from the run's own log. Closing unmerged as planned.

@AojdevStudio
AojdevStudio deleted the test/scorer-proof branch November 8, 2026 12:44
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