Skip to content

Commit 0600621

Browse files
kevingurneysgilmore10kou
authored
GH-49890: [Dev] Group files under component comment headers in .github/CODEOWNERS (#49891)
### Rationale for this change Based on the feedback shared by @ kou in #49811 (comment), we should consider grouping files under component comment headers in `.github/CODEOWNERS` to improve readability. We should also add `@ kevingurney` and `@ sgilmore10` as `CODEOWNERS` for the MATLAB-related CI file `.github/workflows/matlab.yml`. ### Component(s) Developer Tools ### What changes are included in this PR? 1. Group files under component comment headers in `.github/CODEOWNERS` file. 2. Add `@ kevingurney` and `@ sgilmore10` as `CODEOWNERS` for file `.github/workflows/matlab.yml`. ### Are these changes tested? N/A. ### Are there any user-facing changes? No. ### Notes 1. Thank you @ sgilmore10 for your help with this pull request! * GitHub Issue: #49890 Lead-authored-by: Kevin Gurney <kgurney@mathworks.com> Co-authored-by: Kevin Gurney <kevin.p.gurney@gmail.com> Co-authored-by: Sarah Gilmore <sgilmore@mathworks.com> Co-authored-by: Sutou Kouhei <kou@cozmixng.org> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
1 parent c13f3bc commit 0600621

1 file changed

Lines changed: 27 additions & 17 deletions

File tree

.github/CODEOWNERS

Lines changed: 27 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -25,21 +25,6 @@
2525
# Arrow Format
2626
# /format/
2727

28-
## Components
29-
/c_glib/ @kou
30-
# /cpp/
31-
/cpp/src/arrow/acero @westonpace
32-
/cpp/src/arrow/adapters/orc @wgtmac
33-
/cpp/src/arrow/engine @westonpace
34-
/cpp/src/arrow/flight/ @lidavidm
35-
/cpp/src/parquet @wgtmac
36-
/matlab/ @kevingurney @kou @sgilmore10
37-
/python/ @AlenkaF @raulcd @rok
38-
/python/pyarrow/_flight.pyx @lidavidm
39-
/python/pyarrow/**/*gandiva* @wjones127
40-
/r/ @jonkeane @thisisnic
41-
/ruby/ @kou
42-
4328
# Docs
4429
# /docs/
4530
# *.md
@@ -53,16 +38,41 @@
5338
.pre-commit-config.yaml @raulcd
5439
# .git*
5540

56-
# release scripts, archery etc.
41+
# Release scripts, archery etc.
5742
/ci/ @assignUser @jonkeane @kou @raulcd
5843
/dev/ @assignUser @jonkeane @kou @raulcd
5944
.dockerignore @raulcd
6045
.env @assignUser @jonkeane @kou @raulcd
6146
compose.yaml @assignUser @jonkeane @kou @raulcd
6247

63-
# R specific packaging tooling
48+
# Components
49+
50+
## C GLib
51+
/c_glib/ @kou
52+
53+
## C++
54+
/cpp/src/arrow/acero @westonpace
55+
/cpp/src/arrow/adapters/orc @wgtmac
56+
/cpp/src/arrow/engine @westonpace
57+
/cpp/src/arrow/flight/ @lidavidm
58+
/cpp/src/parquet @wgtmac
59+
60+
## MATLAB
61+
/.github/workflows/matlab.yml @kevingurney @sgilmore10
62+
/matlab/ @kevingurney @kou @sgilmore10
63+
64+
## Python
65+
/python/ @AlenkaF @raulcd @rok
66+
/python/pyarrow/_flight.pyx @lidavidm
67+
/python/pyarrow/**/*gandiva* @wjones127
68+
69+
## R
70+
/r/ @jonkeane @thisisnic
6471
/r/configure* @assignUser
6572
/r/Makefile @assignUser
6673
/r/PACKAGING.md @assignUser
6774
/r/tools/ @assignUser
6875
/r/.Rbuildignore @assignUser
76+
77+
## Ruby
78+
/ruby/ @kou

0 commit comments

Comments
 (0)