-
Notifications
You must be signed in to change notification settings - Fork 3.9k
GH-47738: [R] Update NEWS.md for 22.0.0 #47739
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this 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.
under the License. | ||
--> | ||
|
||
# arrow 21.0.0.9000 |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
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. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
There was a problem hiding this comment.
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.
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