Skip to content

ci: raise a PR for rankings refresh instead of pushing to main - #2

Merged
dbut2 merged 1 commit into
mainfrom
rankings-open-pr
Jul 27, 2026
Merged

ci: raise a PR for rankings refresh instead of pushing to main#2
dbut2 merged 1 commit into
mainfrom
rankings-open-pr

Conversation

@dbut2

@dbut2 dbut2 commented Jul 27, 2026

Copy link
Copy Markdown
Owner

The main ruleset requires changes to go through a pull request, so the direct git push at the end of the rankings job was rejected with GH013.

Replaces the hand-rolled commit-and-push with peter-evans/create-pull-request@v8, matching the workflow-sync job in dbut2/x — same action, same bot author/committer, same automerge label. Differences from that job: a reusable rankings branch (repeat runs update the one open PR instead of stacking), and add-paths: README.md since that's all cmd/rankings touches.

The action creates nothing when the rankings table is unchanged, so the explicit git diff --quiet README.md guard is no longer needed.

The main ruleset requires changes to go through a pull request, so the
direct `git push` from the rankings job was rejected with GH013.

Use peter-evans/create-pull-request, matching the workflow-sync job in
dbut2/x: same action, same bot author/committer, same `automerge` label,
and a reusable `rankings` branch so repeat runs update one PR rather than
stacking new ones. The action no-ops when README.md is unchanged, so the
explicit `git diff --quiet` guard is gone.
@dbut2
dbut2 force-pushed the rankings-open-pr branch from a029796 to 1db5699 Compare July 27, 2026 02:02
@dylan-botler
dylan-botler Bot force-pushed the rankings-open-pr branch from 1db5699 to d9e4c95 Compare July 27, 2026 02:04
@dbut2
dbut2 merged commit 1091882 into main Jul 27, 2026
1 check passed
@dbut2
dbut2 deleted the rankings-open-pr branch July 27, 2026 02:07
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