Skip to content

GH-47738: [R] Update NEWS.md for 22.0.0#47739

Merged
raulcd merged 1 commit intoapache:mainfrom
thisisnic:gh-47738-r-news-22
Oct 14, 2025
Merged

GH-47738: [R] Update NEWS.md for 22.0.0#47739
raulcd merged 1 commit intoapache:mainfrom
thisisnic:gh-47738-r-news-22

Conversation

@thisisnic
Copy link
Member

@thisisnic thisisnic commented Oct 7, 2025 •
edited by github-actions bot
Loading

Rationale for this change

Summarise changes for release

What changes are included in this PR?

Update NEWS file

Are these changes tested?

No

Are there any user-facing changes?

No

@thisisnic thisisnic requested a review from jonkeane as a code owner October 7, 2025 12:31
@thisisnic thisisnic added this to the 22.0.0 milestone Oct 7, 2025
@thisisnic thisisnic requested review from amoeba and removed request for jonkeane October 7, 2025 12:32
@github-actions github-actions bot added the awaiting committer review Awaiting committer review label Oct 7, 2025
@thisisnic thisisnic requested a review from jonkeane October 7, 2025 14:25
raulcd
raulcd reviewed Oct 13, 2025
View reviewed changes
Copy link
Member

@raulcd raulcd left a comment

Choose a reason for hiding this comment

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

Do we have to merge this for the release? Or can I proceed, from what I can see I can probably continue with the release today once other blockers are merged.

r/NEWS.md
@@ -19,6 +19,14 @@

# arrow 21.0.0.9000
Copy link
Member

@raulcd raulcd Oct 13, 2025

Choose a reason for hiding this comment

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

is this for arrow 21 or 22? No sure how this works but felt strange to see 21.0.0.9000 here talking about NEWS for arrow 22.

Copy link
Member Author

@thisisnic thisisnic Oct 13, 2025

Choose a reason for hiding this comment

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

It for 22, but the value gets bumped automatically with the release scripts later.

@github-actions github-actions bot added awaiting review Awaiting review awaiting changes Awaiting changes and removed awaiting committer review Awaiting committer review awaiting review Awaiting review awaiting changes Awaiting changes labels Oct 13, 2025
@thisisnic
Copy link
Member Author

thisisnic commented Oct 13, 2025

Not a blocker, we can cherry pick it into the R package release branch later if we need. You're welcome to approve/merge as-is though; doesn't really need the others to look it over.

raulcd
raulcd approved these changes Oct 13, 2025
View reviewed changes
Copy link
Member

@raulcd raulcd left a comment

Choose a reason for hiding this comment

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

👍

@github-actions github-actions bot added awaiting merge Awaiting merge and removed awaiting changes Awaiting changes labels Oct 13, 2025
amoeba
amoeba approved these changes Oct 13, 2025
View reviewed changes
Copy link
Member

@amoeba amoeba left a comment •
edited
Loading

Choose a reason for hiding this comment

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

+1 thanks @thisisnic. I was out last week.

@raulcd raulcd merged commit 0f311e1 into apache:main Oct 14, 2025
15 checks passed
@raulcd raulcd removed the awaiting merge Awaiting merge label Oct 14, 2025
@raulcd raulcd mentioned this pull request Oct 14, 2025
Closed
raulcd pushed a commit that referenced this pull request Oct 14, 2025
@thisisnic @raulcd
GH-47738: [R] Update NEWS.md for 22.0.0 (#47739)
2140934 
### Rationale for this change

Summarise changes for release

### What changes are included in this PR?

Update NEWS file

### Are these changes tested?

No

### Are there any user-facing changes?

No
* GitHub Issue: #47738

Authored-by: Nic Crane <[email protected]>
Signed-off-by: Raúl Cumplido <[email protected]>
@conbench-apache-arrow
Copy link

conbench-apache-arrow bot commented Oct 14, 2025

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

There were no benchmark performance regressions. 🎉

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

zanmato1984 pushed a commit to zanmato1984/arrow that referenced this pull request Oct 15, 2025
@thisisnic @zanmato1984
apacheGH-47738: [R] Update NEWS.md for 22.0.0 (apache#47739)
dba3e9a 
### Rationale for this change

Summarise changes for release

### What changes are included in this PR?

Update NEWS file

### Are these changes tested?

No

### Are there any user-facing changes?

No
* GitHub Issue: apache#47738

Authored-by: Nic Crane <[email protected]>
Signed-off-by: Raúl Cumplido <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@amoeba amoeba amoeba approved these changes

@raulcd raulcd raulcd approved these changes

@jonkeane jonkeane Awaiting requested review from jonkeane jonkeane is a code owner

Assignees

No one assigned

Labels

Component: Documentation Component: R

Projects

None yet

Milestone

22.0.0

Development

Successfully merging this pull request may close these issues.

3 participants

@thisisnic @amoeba @raulcd