Skip to content

MINOR: [Dev] Add pyarrow codeowners - #46176

Merged
kou merged 3 commits into
apache:mainfrom
raulcd:add-pyarrow-codeowners
Apr 17, 2025
Merged

MINOR: [Dev] Add pyarrow codeowners#46176
kou merged 3 commits into
apache:mainfrom
raulcd:add-pyarrow-codeowners

Conversation

@raulcd

@raulcd raulcd commented Apr 17, 2025

Copy link
Copy Markdown
Member

Rationale for this change

Sometimes we miss some Python PRs that we could potentially review.

What changes are included in this PR?

Add some of us to codeowners Python folder to get pinged on PRs.

Are these changes tested?

No

Are there any user-facing changes?

No

@github-actions github-actions Bot added the awaiting committer review Awaiting committer review label Apr 17, 2025
Comment thread .github/CODEOWNERS Outdated
@github-actions github-actions Bot added awaiting changes Awaiting changes and removed awaiting committer review Awaiting committer review labels Apr 17, 2025
Co-authored-by: Alenka Frim <AlenkaF@users.noreply.github.com>
@github-actions github-actions Bot added awaiting change review Awaiting change review and removed awaiting changes Awaiting changes labels Apr 17, 2025
Comment thread .github/CODEOWNERS Outdated

@kou kou 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.

+1

@kou
kou merged commit c2ec1cd into apache:main Apr 17, 2025
@github-actions github-actions Bot added awaiting changes Awaiting changes awaiting merge Awaiting merge and removed awaiting change review Awaiting change review awaiting changes Awaiting changes labels Apr 17, 2025
@conbench-apache-arrow

Copy link
Copy Markdown

After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit c2ec1cd.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about 2 possible false positives for unstable benchmarks that are known to sometimes produce them.

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

Labels

awaiting merge Awaiting merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants