kevingurney commented on PR #49891: URL: https://github.com/apache/arrow/pull/49891#issuecomment-4353352287
After looking more closely at https://github.com/apache/arrow/pull/49891#discussion_r3165078983 and reading https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners#example-of-a-codeowners-file, it seems like the location of the pattern `/.github/workflows/matlab.yml` is problematic here. My understanding is that `/.github/workflows/matlab.yml` would have to come after the "more general pattern" of `/.github/` in order for the "correct" code owners to be notified. This doesn't exactly feel ideal since the MATLAB CI pattern will no longer be associated with the MATLAB component header. However, there probably isn't anything we can really do here, so I am going to update this PR to move the MATLAB CI pattern accordingly. Any flags with this change @kou? -- 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]
