Skip to content

GH-55259: [Docs] Add guidelines for reviewers on AI generated PRs - #55260

Open
raulcd wants to merge 4 commits into
apache:mainfrom
raulcd:GH-55259
Open

raulcd wants to merge 4 commits into
apache:mainfrom
raulcd:GH-55259

Conversation

@raulcd

@raulcd raulcd commented Oct 26, 2026 •

Copy link
Copy Markdown
Member

Rationale for this change

Add documentation of our practices for AI generated code/review comments.

What changes are included in this PR?

Some documentation with a proposed message and indications about possibility of closing PRs or adding labels indicating this might be AI-slop or requires a human in the loop.

Are these changes tested?

No only docs.

Are there any user-facing changes?

No

Was AI used for this PR?

In accordance to the AI generation guidelines, please disclose below whether and how AI was used in this PR.

PR code and description written by:

  • Human
  • AI

Reviewed before submission by:

  • Human
  • AI
  • Not reviewed

@raulcd

raulcd commented Oct 26, 2026

Copy link
Copy Markdown
Member Author

@github-actions crossbow submit preview-docs

@github-actions github-actions Bot added the awaiting committer review Awaiting committer review label Oct 26, 2026
@github-actions

Copy link
Copy Markdown

⚠️ GitHub issue #55259 has been automatically assigned in GitHub to PR creator.

@github-actions

This comment was marked as outdated.

@raulcd

raulcd commented Oct 26, 2026

Copy link
Copy Markdown
Member Author

@github-actions crossbow submit preview-docs

@github-actions

This comment was marked as outdated.

@raulcd
raulcd marked this pull request as ready for review October 26, 2026 12:26
Copilot AI balanced review requested due to automatic review settings October 26, 2026 12:26
@raulcd

raulcd commented Oct 26, 2026

Copy link
Copy Markdown
Member Author

Copilot AI 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.

🟢 Approval recommended

The documentation changes address the requested reviewer guidance with no unresolved blocking issues.

No open issues 🎉

What changed in this PR

This documentation-only PR extends Arrow’s AI contribution guidelines with expectations for human participation during review, addressing #55259.

Changes:

  • Clarifies that contributors should engage directly with reviewers.
  • Documents possible closure or labeling of PRs showing excessive reliance on AI.
  • Adds an example reviewer message linking to the AI-generated code policy.
File Description
docs/​source/​developers/​overview.rst Adds human-engagement expectations and guidance for responding to excessive AI usage.

Review effort: Balanced

@AlenkaF AlenkaF left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks! Just one nit connected to the dropdown box.

Comment thread docs/source/developers/overview.rst Outdated
Co-authored-by: Alenka Frim <AlenkaF@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings October 26, 2026 14:06

Copilot AI 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.

🟢 Approval recommended

The focused documentation changes match the stated purpose and have no identified blocking issues.

No open issues 🎉

Review effort: Balanced

@raulcd

raulcd commented Oct 26, 2026

Copy link
Copy Markdown
Member Author

@github-actions crossbow submit preview-docs

@github-actions

Copy link
Copy Markdown

Revision: 30347ce

Submitted crossbow builds: ursacomputing/crossbow @ actions-be29f0bb4a

Task Status
preview-docs GitHub Actions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants