Skip to content

Conversation

@raulcd
Copy link
Member

@raulcd raulcd commented May 8, 2024

Rationale for this change

Dane is a new committer.

What changes are included in this PR?

Remove the collaborator list.

Are these changes tested?

Not required.

Are there any user-facing changes?

No

@raulcd raulcd requested review from assignUser and kou as code owners May 8, 2024 10:19
@github-actions github-actions bot added the awaiting committer review Awaiting committer review label May 8, 2024
@raulcd
Copy link
Member Author

raulcd commented May 8, 2024

@danepitkin this is something you can probably use as a first merge. See the details to merge here: https://github.com/apache/arrow/tree/main/dev#how-to-merge-a-pull-request

@raulcd raulcd requested a review from danepitkin May 8, 2024 10:20
@danepitkin
Copy link
Member

Thanks, Raul! Great idea ;)

@danepitkin danepitkin merged commit 318d22a into apache:main May 8, 2024
@danepitkin danepitkin removed the awaiting committer review Awaiting committer review label May 8, 2024
@danepitkin
Copy link
Member

🎉

@conbench-apache-arrow
Copy link

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

There were no benchmark performance regressions. 🎉

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

vibhatha pushed a commit to vibhatha/arrow that referenced this pull request May 25, 2024
### Rationale for this change

Dane is a new committer.

### What changes are included in this PR?

Remove the collaborator list.

### Are these changes tested?

Not required.

### Are there any user-facing changes?

No

Authored-by: Raúl Cumplido <raulcumplido@gmail.com>
Signed-off-by: Dane Pitkin <dane@voltrondata.com>
@danepitkin
Copy link
Member

danepitkin commented Jun 11, 2024

@raulcd How can I regain access to github actions? It seems I lost it: #42113 (comment)

@assignUser
Copy link
Member

@danepitkin
Copy link
Member

Thank you!!

Sign up for free to 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