Skip to content

docs: Clarify assisted contribution expectations - #4427

Merged
gmlewis merged 1 commit into
google:masterfrom
tomatotomata:codex/contributor-review-guidance-3939
Aug 4, 2026
Merged

docs: Clarify assisted contribution expectations#4427
gmlewis merged 1 commit into
google:masterfrom
tomatotomata:codex/contributor-review-guidance-3939

Conversation

@tomatotomata

@tomatotomatatomatotomata commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add a short section for contributors using code or writing assistants
  • ask contributors to disclose assistance and review the rendered diff and description
  • reinforce focused, actionable review communication

Verification

  • git diff --check
  • documentation-only change

Assistance disclosure

I used an LLM to help draft the wording, then reviewed and edited the final text and verified the rendered diff manually.

Relates to: #3939

Signed-off-by: ahmadalguydi <ahmadalgaidy@hotmail.com>
@gmlewisgmlewis changed the title docs: clarify assisted contribution expectationsdocs: Clarify assisted contribution expectationsAug 2, 2026
@gmlewisgmlewis added the NeedsReview PR is awaiting a review before merging. label Aug 2, 2026
@codecov

codecovBot commented Aug 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.54%. Comparing base (e496a87) to head (fbe54ce).

Additional details and impacted files
@@ Coverage Diff @@## master #4427 +/- ##
=======================================
Coverage 97.54% 97.54% =======================================
Files 194 194 Lines 19836 19836 =======================================
Hits 19350 19350 + Misses 269 268 -1 - Partials 217 218 +1 

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@gmlewisgmlewis left a comment

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.

Although some of this is mentioned in the "Tips" section above it, I'm OK with adding this, but will NOT be closing the issue referred to in the description, as I have found that particular issue to be extremely useful to keep open.
LGTM.

cc: @stevehipwell - @alexandear - @Not-Dhananjay-Mishra

@gmlewisgmlewis removed the NeedsReview PR is awaiting a review before merging. label Aug 4, 2026
@gmlewis
gmlewis merged commit 29207ef into google:masterAug 4, 2026
15 checks passed
Sign up for freeto 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.

3 participants

@tomatotomata@gmlewis@Not-Dhananjay-Mishra