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

@sweetmantech sweetmantech 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/chat#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/chat#1841 process cleanup.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@vercel

vercel Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated (UTC)
api Ready Ready Preview Jul 3, 2026 8:32pm

Request Review

@coderabbitai

coderabbitai Bot 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-ai cubic-dev-ai 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 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 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