Skip to content

MINOR: [R] Add 25.0.1 NEWS entry - #50954

Open
thisisnic wants to merge 1 commit into
apache:mainfrom
thisisnic:minor-r-news-25.0.1
Open

MINOR: [R] Add 25.0.1 NEWS entry#50954
thisisnic wants to merge 1 commit into
apache:mainfrom
thisisnic:minor-r-news-25.0.1

Conversation

@thisisnic

Copy link
Copy Markdown
Member

Rationale for this change

The 25.0.1 section of r/NEWS.md on main is empty; the CRAN 25.0.1 release branch has an entry for the mimalloc change that shipped in the patch release.

What changes are included in this PR?

Adds the 25.0.1 NEWS entry to match the CRAN release branch.

Are these changes tested?

No, documentation only.

Are there any user-facing changes?

No.

@thisisnic
thisisnic requested a review from jonkeane as a code owner August 22, 2026 16:17
Copilot AI lite review requested due to automatic review settings August 22, 2026 16:17
@github-actions github-actions Bot added the awaiting committer review Awaiting committer review label Aug 22, 2026

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.

Pull request overview

Adds the missing Arrow R 25.0.1 NEWS entry documenting the macOS mimalloc improvement.

Changes:

  • Adds the “Minor improvements and fixes” section.
  • References issue #50428.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@uros-b

uros-b commented Aug 24, 2026

Copy link
Copy Markdown
Member

Nice!

@thisisnic

Copy link
Copy Markdown
Member Author

@uros-b Hey there, I've seen you approving various PRs across different areas of the codebase. Just out of curiosity, what's the reason for this?

I have concerns that it could either a) cause confusion to new contributors who need their PRs reviewed by a committer and could mix up an approving review by a non-committer with an intention to merge by a committer, or b) lead to committers not reviewing/merging PRs as at first glance it looks like another committer has already reviewed it.

I don't think it's technically disallowed, but it's very unusual so I'm curious as to the reasoning.

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