Skip to content

docs: PRs target main; retire the test-branch PR flow - #750

Merged
sweetmantech merged 2 commits into
mainfrom
chore/prs-target-main
Jul 3, 2026
Merged

docs: PRs target main; retire the test-branch PR flow#750
sweetmantech merged 2 commits into
mainfrom
chore/prs-target-main

Conversation

@sweetmantech

@sweetmantechsweetmantech commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

What

PRs in this repo now target main. The test branch is retired as a PR target; the sync-test-with-main task-start ritual is removed from the agent instructions.

Why

Requested by @sweetmantech 2026-07-03 during recoupable/app#1841: four PRs were opened against test because these instructions mandated it; the convention is now main-only. Sibling PRs update the other repos' instructions and the dev skills (recoupable/skills#68).

🤖 Generated with Claude Code


Summary by cubic

Set main as the PR base and retire the test branch flow. Removes the start-of-task test sync step to avoid mis-targeted PRs and simplify agent instructions.

  • Migration
    • Branch from and open PRs against main (e.g., gh pr create --base main).
    • Stop using the test branch and the old sync-with-main ritual.

Written for commit 846dd98. Summary will update on new commits.

Review in cubic

All PRs now open against main. The test branch is no longer a PR
target and the sync-test-with-main task-start ritual is removed.
Context: recoupable/app#1841 process cleanup.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@vercel

vercelBot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

ProjectDeploymentActionsUpdated (UTC)
apiReadyReadyPreviewJul 3, 2026 8:32pm

Request Review

@coderabbitai

coderabbitaiBot commented Jul 3, 2026

Copy link
Copy Markdown

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • AGENTS.md is excluded by none and included by none

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 58b003ad-b80a-431d-88f5-2681f933d429

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/prs-target-main

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.

@cubic-dev-aicubic-dev-aiBot 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 issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.
Architecture diagram
sequenceDiagram
participant Dev as Developer
participant Agent as Claude Code Agent
participant GH as GitHub
participant Local as Local Git Repo
Note over Dev,Local: NEW: test branch retired – PRs target main directly
Dev->>Agent: Start new task
Agent->>Local: git checkout main && git pull origin main
Agent->>Local: git checkout -b feature-branch
Agent-->>Dev: Ready to work
Note over Agent,GH: Push commits and open PR
Agent->>Local: git push -u origin feature-branch
Agent->>GH: gh pr create --base main
GH-->>Agent: PR URL
Agent-->>Dev: PR ready – wait for approval
Loading

Auto-approved: Updates AGENTS.md to target main branch and remove test-branch workflow. Only documentation change, no code or logic modified.

Re-trigger cubic

Sign up for freeto 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

@sweetmantech