Skip to content

docs: condense contributing review notes#792

Merged
mfoltz merged 1 commit intomainfrom
codex/edit-contributing.md-to-condense-review-notes-2026-03-22t11-31-54z
Mar 22, 2026
Merged

docs: condense contributing review notes#792
mfoltz merged 1 commit intomainfrom
codex/edit-contributing.md-to-condense-review-notes-2026-03-22t11-31-54z

Conversation

@mfoltz
Copy link
Copy Markdown
Owner

@mfoltz mfoltz commented Mar 22, 2026

Motivation

  • Keep the release-process rules and branch/channel policy intact while removing reviewer-only troubleshooting details to keep contributor-facing docs concise and focused.

Description

  • Edit .github/CONTRIBUTING.md to preserve the ## Release process section unchanged and replace the ## Review/testing note with two short bullets: run bash .codex/install.sh before local checks and do not run bash -n on .github/workflows/*.yml, removing the inline Python/PyYAML examples and the ModuleNotFoundError troubleshooting guidance.

Testing

  • Ran git diff --check, which completed with no issues.

Codex Task

Copilot AI review requested due to automatic review settings March 22, 2026 11:31
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR condenses contributor-facing guidance by trimming reviewer-specific troubleshooting details from the contributing docs while keeping the release policy documentation intact.

Changes:

  • Preserved the existing ## Release process section unchanged.
  • Replaced the detailed ## Review/testing note troubleshooting content with two concise bullets focused on environment setup and correct validation scope.

@mfoltz mfoltz merged commit fab1364 into main Mar 22, 2026
10 checks passed
@mfoltz mfoltz deleted the codex/edit-contributing.md-to-condense-review-notes-2026-03-22t11-31-54z branch March 22, 2026 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants