Skip to content

Prepare bpm-release for the main default branch - #237

Merged
aramprice merged 1 commit into
masterfrom
codex/eight-default-branches-main
Nov 3, 2026
Merged

aramprice merged 1 commit into
masterfrom
codex/eight-default-branches-main

Conversation

@aramprice

@aramprice aramprice commented Nov 3, 2026 •

Copy link
Copy Markdown
Member

Change both own Git sources to main and update contributor guidance. Version counters, release tags and outbound references remain unchanged. Documentation and comment follow-ups are separate.

Merge the source preparation before the native default-branch rename. Keep live pipeline configuration unchanged until main exists; pause/drain repository and version writers, rename, reconfigure and verify actual sources, then restore the original pause states. Do not publish releases or advance counters solely for validation.

Validation: exact parsed comparison confirms only the intended own Git source branches change in the pipeline; all other fields are preserved. Strict Concourse validation passes before and after. git diff --check passes.

Migration tracker: #238. This preparation PR does not close it. Shared community configuration: cloudfoundry/community#1680 (draft until all eight cutovers are verified).

Documentation/comment follow-ups:

Update both own Git sources and contributor guidance.
@coderabbitai

coderabbitai Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Walkthrough

The pull-request instructions now specify main as the target branch. The bpm-release and bpm-ci Git resources in the CI pipeline now track main instead of master.

Suggested reviewers: mkocher

Merge Risk: Low · up to 9dd6b

The preparation is mergeable with awareness that contributor instructions will temporarily name a nonexistent branch. Keep the guidance accurate until the rename, or explicitly document the transition.

Pre-merge checks: Passed 4
Check name Status Explanation
Title check Passed The title states the main change: preparing the bpm-release repository for a main default branch. The raw summary shows that CONTRIBUTING.md and ci/pipeline.yml now point to main instead of master, wh…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Description check Passed The description directly matches the changeset. It identifies the two Git sources changed to main, the contributor guidance update, preserved configuration, rollout procedure, and validation results…
Full details: Title check

Explanation

The title states the main change: preparing the bpm-release repository for a main default branch. The raw summary shows that CONTRIBUTING.md and ci/pipeline.yml now point to main instead of master, which matches the title's subject. The title is specific and concise, and it names the primary change.

Full details: Description check

Explanation

The description directly matches the changeset. It identifies the two Git sources changed to main, the contributor guidance update, preserved configuration, rollout procedure, and validation results.

Finishing Touches
Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • 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

CodeRabbit Verify

  • Run CodeRabbit Verify to test the changes in this pull request.
Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 52d32b7c-5247-4cd1-a1a5-8e1e55ea663d
Commits

Reviewing files that changed from the base of the PR and between 6cd5d8c and 9dd6bba.

Files selected for processing: 2
  • CONTRIBUTING.md
  • ci/pipeline.yml

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

@coderabbitai coderabbitai 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.

  • Fix comments: Fix CodeRabbit comments on this PR

Comment thread CONTRIBUTING.md Resolved
@aramprice
aramprice merged commit e61630e into master Nov 3, 2026
4 checks passed
@aramprice
aramprice deleted the codex/eight-default-branches-main branch November 3, 2026 00:59
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