Skip to content

docs(claude): stop the daily maintenance PR pileup#42

Merged
bryanfawcett merged 1 commit into
mainfrom
claude/sleepy-albattani-whplnt
Jul 3, 2026
Merged

docs(claude): stop the daily maintenance PR pileup#42
bryanfawcett merged 1 commit into
mainfrom
claude/sleepy-albattani-whplnt

Conversation

@bryanfawcett

Copy link
Copy Markdown
Contributor

Summary

The scheduled "keep the repo up to date" routine had been opening a new draft PR every day for the past 8 days (#30, #31, #33, #34, #35, #36, #37, #39) without checking for existing open PRs or ever merging its own work, and repeatedly hand-resolved actions/checkout SHA drift in inconsistent directions (flip-flopping between two SHAs) instead of deferring to Dependabot. This adds explicit rules to CLAUDE.md for scheduled/unattended sessions: dedupe against open PRs from the same routine before opening a new one, leave third-party Action version pins to Dependabot (which already owns that job weekly), run formatters before committing, and merge routine/mechanical fixes once required checks are green instead of leaving them open indefinitely.

Linked issue(s)

Closes #

Type of change

  • docs — documentation only

Breaking change?

  • This PR introduces a breaking change.

How has this been tested?

Ran prettier --check and markdownlint-cli2 locally against the changed file; both pass. This is a documentation-only change with no runtime surface to exercise.

Checklist

Process

  • My PR title follows Conventional Commits.
  • All of my commits are signed (SSH) and show "Verified" on GitHub.
  • All of my commits have a DCO sign-off (Signed-off-by: trailer, added by git commit -s).
  • My branch name follows claude/<short-kebab-description>.
  • I have rebased onto the latest default branch; no merge commits from the base branch in this PR.
  • I have added or updated tests covering the change — N/A, documentation only.
  • I have updated documentation where relevant (this PR is the documentation update).
  • I ran the repo's local checks (prettier, markdownlint) and they pass.
  • Any new runtime dependencies are justified in this PR description — N/A, none added.
  • If this is a breaking change... — N/A.

NA-03 merge blockers

  • N/A — this change does not touch schema, APIs, dependencies, cryptography, data placement, or smart-contract counts.

Additional notes

Companion cleanup done in this session (not part of this diff):


Generated by Claude Code

The scheduled "keep the repo up to date" routine was opening a new
draft PR every day (#30-#39) without checking for existing open ones
or merging its own work, and repeatedly hand-resolved actions/checkout
SHA drift in inconsistent directions instead of deferring to
Dependabot. Document the actual rules: dedupe against open PRs from
the same routine, leave third-party Action version pins to Dependabot,
run formatters before committing, and merge routine/mechanical fixes
once required checks are green instead of leaving them as drafts
forever.

Signed-off-by: Claude <noreply@anthropic.com>
@bryanfawcett bryanfawcett merged commit bdcd780 into main Jul 3, 2026
8 checks passed
@bryanfawcett bryanfawcett deleted the claude/sleepy-albattani-whplnt branch July 3, 2026 05:17
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