Skip to content

docs(contributing): harvest-credit close template + stewardship branch explanation#2986

Open
Hmbown wants to merge 1 commit into
mainfrom
docs/harvest-template-stewardship
Open

docs(contributing): harvest-credit close template + stewardship branch explanation#2986
Hmbown wants to merge 1 commit into
mainfrom
docs/harvest-template-stewardship

Conversation

@Hmbown

@Hmbown Hmbown commented Jun 10, 2026

Copy link
Copy Markdown
Owner

Two process additions from the 2026-06-09 triage session, for owner review:

  1. Harvest-credit close template — harvested PRs are closed with a comment naming the contributor, the exact landed commits/PRs, and a tracking issue for any remainder (the pattern feat: porting to HarmonyOS #2634 got right, now written down). Never a bare "superseded".
  2. Stewardship branch section — documents codex/v0.9.0-stewardship's role so external contributors base layered refactor series correctly (as @aboimpinto already does with refactor(commands): extract registry and parser helpers #2888).

Refs #2985 (release-flow hygiene, filed separately).

🤖 Generated with Claude Code

…h explanation

Encodes two process decisions from the 2026-06-09 triage:
- harvested PRs are closed with a comment naming the contributor, the
  landed commits/PRs, and a tracking issue for any remainder (the #2634
  pattern, now the template)
- documents codex/v0.9.0-stewardship's role so external contributors
  base layered refactor series correctly (as #2888 already does)

Refs #2985.

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

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the CONTRIBUTING.md file to document the template for closing harvested PRs by hand and introduces the "Stewardship Branch" workflow for staging large refactors. The review feedback suggests replacing the hardcoded version v0.9.0 in the branch name with a generic placeholder (e.g., codex/v<version>-stewardship) to prevent the documentation from quickly becoming outdated.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread CONTRIBUTING.md
Comment on lines +191 to +192
Large refactors and architecture work stage on
`codex/v0.9.0-stewardship` before reaching `main`. The branch exists so

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

Hardcoding the specific version v0.9.0 in the branch name will cause this documentation to quickly become outdated as the project progresses. Consider using a placeholder like codex/v<version>-stewardship to keep the guide maintainable across future releases.

Suggested change
Large refactors and architecture work stage on
`codex/v0.9.0-stewardship` before reaching `main`. The branch exists so
Large refactors and architecture work stage on
`codex/v<version>-stewardship` (e.g., `codex/v0.9.0-stewardship`) before reaching `main`. The branch exists so

Comment thread CONTRIBUTING.md

What this means for you:

- **Base layered/EPIC-sized refactor PRs on `codex/v0.9.0-stewardship`**

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

Use the placeholder version here as well to maintain consistency with the section introduction.

Suggested change
- **Base layered/EPIC-sized refactor PRs on `codex/v0.9.0-stewardship`**
- **Base layered/EPIC-sized refactor PRs on `codex/v<version>-stewardship`**

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