Copilot commented on code in PR #49891: URL: https://github.com/apache/arrow/pull/49891#discussion_r3165079004
########## .github/CODEOWNERS: ########## @@ -25,19 +25,36 @@ # Arrow Format # /format/ -## Components +# Components + +## C Glib Review Comment: Component header uses `C Glib`, but the project consistently refers to this component as `C GLib` (e.g., `.github/workflows/ruby.yml` and `r/_pkgdown.yml`). Consider updating the header capitalization for consistency and searchability. ```suggestion ## C GLib ``` ########## .github/CODEOWNERS: ########## @@ -25,19 +25,36 @@ # Arrow Format # /format/ -## Components +# Components + +## C Glib /c_glib/ @kou -# /cpp/ + +## C++ /cpp/src/arrow/acero @westonpace /cpp/src/arrow/adapters/orc @wgtmac /cpp/src/arrow/engine @westonpace /cpp/src/arrow/flight/ @lidavidm /cpp/src/parquet @wgtmac + +## MATLAB /matlab/ @kevingurney @kou @sgilmore10 +/.github/workflows/matlab.yml @kevingurney @sgilmore10 Review Comment: `/.github/workflows/matlab.yml` is currently listed *before* the broader `/.github/ ...` rule later in the file. In CODEOWNERS, the last matching pattern wins, so the later `/.github/` entry will override this and the MATLAB-specific owners won't be applied. Move the `/.github/workflows/matlab.yml` rule below the `/.github/` rule (or otherwise narrow the `/.github/` rule) so it takes precedence for that file. -- 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]
