Copilot commented on code in PR #50497:
URL: https://github.com/apache/arrow/pull/50497#discussion_r3570019463


##########
r/NEWS.md:
##########
@@ -21,11 +21,45 @@
 
 # arrow 25.0.0
 
+## Breaking changes
+
 - Arrow `uint64` types are now always converted to R `double` (numeric) 
vectors,
   regardless of the values. Previously, small `uint64` values were converted to
   R `integer`, which could cause inconsistent types within list columns when
   different list elements had different value ranges (#50339).
 
+## New features
+
+- `Field` objects now support field-level metadata via `$metadata` and
+  `$with_metadata()` (@max-romagnoli, #33390).
+- Parquet files now support list-columns of ordered factors (ordered
+  dictionaries) (#49689).
+
+## Minor improvements and fixes
+
+- `Array$create()` now gives a clearer error message when given a `POSIXct`
+  object with an invalid timezone (#40886).
+- Dictionary arrays with `large_string` value types now convert correctly to R
+  factors (#39603).
+- `open_dataset()` now gives a clearer error message when providing a mix of
+  readr and Arrow options (@Rich-T-kid, #33420).

Review Comment:
   For consistency with other code identifiers in this section (e.g., 
`dbplyr`), the package name should be formatted as inline code.



##########
r/NEWS.md:
##########
@@ -21,11 +21,45 @@
 
 # arrow 25.0.0
 
+## Breaking changes
+
 - Arrow `uint64` types are now always converted to R `double` (numeric) 
vectors,
   regardless of the values. Previously, small `uint64` values were converted to
   R `integer`, which could cause inconsistent types within list columns when
   different list elements had different value ranges (#50339).
 
+## New features
+
+- `Field` objects now support field-level metadata via `$metadata` and
+  `$with_metadata()` (@max-romagnoli, #33390).
+- Parquet files now support list-columns of ordered factors (ordered
+  dictionaries) (#49689).
+
+## Minor improvements and fixes
+
+- `Array$create()` now gives a clearer error message when given a `POSIXct`
+  object with an invalid timezone (#40886).
+- Dictionary arrays with `large_string` value types now convert correctly to R
+  factors (#39603).
+- `open_dataset()` now gives a clearer error message when providing a mix of
+  readr and Arrow options (@Rich-T-kid, #33420).
+- `read_parquet()` no longer triggers a C++ alignment warning from the Acero
+  source node (#46178).
+- `Schema` metadata partial matching on `$metadata$r` no longer errors when
+  other metadata keys start with "r" (#50163).
+- `to_arrow()` now preserves `group_by()` when converting from a dbplyr lazy
+  table (#40640).
+- `write_parquet()` now correctly validates that `max_rows_per_group` is a
+  positive number (#40742).
+- Stale S3 connections no longer cause a segfault during garbage collection
+  (#50009).
+- Spurious "Invalid metadata$r" warnings are no longer emitted when reading
+  files with custom schema metadata (#48712).
+
+## Installation
+
+- The R package now builds under r-universe/r-wasm (#49981).

Review Comment:
   Consider formatting the repository path as inline code to improve 
readability and match other identifier formatting in this NEWS section.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to