Skip to content

Document Codex git commit attribution config#21379

Merged
henzelmann-oai merged 1 commit into
mainfrom
codex/document-commit-attribution
May 6, 2026
Merged

Document Codex git commit attribution config#21379
henzelmann-oai merged 1 commit into
mainfrom
codex/document-commit-attribution

Conversation

@henzelmann-oai

Copy link
Copy Markdown
Contributor

Summary

  • document that commit attribution for generated git commit messages is gated by the codex_git_commit feature flag
  • add an example config.toml snippet showing commit_attribution with [features].codex_git_commit = true
  • update the config schema description so the reference docs explain that commit_attribution only takes effect when the feature is enabled

Fixes #19799.

Validation

  • cargo run -p codex-core --bin codex-write-config-schema
  • cargo test -p codex-config
  • cargo test -p codex-features
  • cargo fmt --check
  • git diff --check

Notes

  • cargo test -p codex-core config_schema_matches_fixture currently fails before reaching the schema test because core_test_support imports similar without a linked crate in this checkout. The narrower package checks above avoid that unrelated test-support build failure.

@henzelmann-oai henzelmann-oai requested a review from a team as a code owner May 6, 2026 17:01
@github-actions

github-actions Bot commented May 6, 2026

Copy link
Copy Markdown
Contributor

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@henzelmann-oai henzelmann-oai force-pushed the codex/document-commit-attribution branch from cdced97 to 793eb62 Compare May 6, 2026 19:13
@henzelmann-oai

Copy link
Copy Markdown
Contributor Author

I have read the CLA Document and I hereby sign the CLA

github-actions Bot added a commit that referenced this pull request May 6, 2026
Comment thread docs/config.md

For a full configuration reference, see [this documentation](https://developers.openai.com/codex/config-reference).

## Commit attribution

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

should this be in For a full configuration reference, see this documentation. ?

@henzelmann-oai henzelmann-oai merged commit 8f5d68f into main May 6, 2026
37 of 39 checks passed
@henzelmann-oai henzelmann-oai deleted the codex/document-commit-attribution branch May 6, 2026 21:14
@github-actions github-actions Bot locked and limited conversation to collaborators May 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clarify Codex commit co-author attribution behavior

2 participants