Skip to content

MINOR: [Docs] Add links to AI-generated code guidance#49131

Merged
thisisnic merged 2 commits intoapache:mainfrom
thisisnic:ai-link
Feb 4, 2026
Merged

MINOR: [Docs] Add links to AI-generated code guidance#49131
thisisnic merged 2 commits intoapache:mainfrom
thisisnic:ai-link

Conversation

@thisisnic
Copy link
Member

Rationale for this change

Add link to AI-generated code guidance - we should make sure the docs are updated before we merge this though

What changes are included in this PR?

Add link to AI-generated code guidance

Are these changes tested?

No

Are there any user-facing changes?

No

Copy link
Member

@pitrou pitrou left a comment

Choose a reason for hiding this comment

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

This looks good to me, though we may also want to make the mention more prominent.

@github-actions github-actions bot added awaiting changes Awaiting changes and removed awaiting committer review Awaiting committer review labels Feb 3, 2026
Co-authored-by: Raúl Cumplido <raulcumplido@gmail.com>
@github-actions github-actions bot added awaiting change review Awaiting change review and removed awaiting changes Awaiting changes labels Feb 3, 2026
Copy link
Member

@kou kou left a comment

Choose a reason for hiding this comment

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

+1

@github-actions github-actions bot added awaiting merge Awaiting merge and removed awaiting change review Awaiting change review labels Feb 4, 2026
@thisisnic
Copy link
Member Author

@pitrou Happy to make the change; when you say more prominent, what do you suggest we do?

@pitrou
Copy link
Member

pitrou commented Feb 4, 2026

Perhaps some specific admonition in bold? But the current solution may be good enough, we'll see if people actually read it...

@thisisnic thisisnic merged commit f895512 into apache:main Feb 4, 2026
10 checks passed
@thisisnic thisisnic removed the awaiting merge Awaiting merge label Feb 4, 2026
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.

4 participants