Conversation
The comment posted by the dev_pr title check pointed at CONTRIBUTING.md#Minor-Fixes, but the heading slug is lowercase (#minor-fixes), so the anchor landed at the top of the page. The conditional clause also ended with a period, producing a sentence fragment; join it with a comma. Signed-off-by: simpleqt <89645338+simpleqt@users.noreply.github.com>
github-actions
Bot
added
Component: Documentation
awaiting review
Reranko05
left a comment
Reranko05
left a comment
There was a problem hiding this comment.
I checked the existing #Minor-Fixes link in title_check.md, and it already navigates to the correct Minor Fixes section on the CONTRIBUTING.md page. Could you clarify if there's a specific case where the current anchor doesn't work?
|
This is an AI-slop. |
|
This AI slop bot has also created a couple other |
|
@raulcd Perhaps open our own PRs for these issues? The main contribution was in finding them, probably, since the fixes are trivial. |
|
@pitrou sure, it was more about what approach to take on those. I am fine not merging them and pushing them ourselves. |
### Rationale for this change Fix three minor issues as discussed in #51358 (comment) ### What changes are included in this PR? - Fix the punctuation in title-check comment. - Remove an extra parenthesis from the `pyrunfile` link in the MATLAB documentation. - Fix the `buffers` parameter name in the `buffers_to_array` documentation. ### Are these changes tested? These are minor documentation fixes. ### Are there any user-facing changes? Yes. ### Was AI used for this PR? In accordance to the [AI generation guidelines](https://arrow.apache.org/docs/dev/developers/overview.html#ai-generated-code), please disclose below whether and how AI was used in this PR. **PR code and description written by:** - [X] Human - [ ] AI **Reviewed before submission by:** - [X] Human - [ ] AI - [ ] Not reviewed Authored-by: Aaditya Srinivasan <aadityasri03@gmail.com> Signed-off-by: Antoine Pitrou <antoine@python.org>
The comment posted verbatim by
.github/workflows/dev_pr/title_check.jswhen a PR title fails the check has two small defects:CONTRIBUTING.md#Minor-Fixes, but the heading's slug is#minor-fixes(GitHub anchors are lowercase), so it lands at the top of the page instead of the Minor Fixes section;Was AI used for this PR?
title_check.js:31) before submitting.