getsentry / sentry

Developer-first error tracking and performance monitoring
https://sentry.io
Other
38.82k stars 4.16k forks source link

CODEOWNERs doesn't support excluding subdirectories #72480

Open mrduncan opened 3 months ago

mrduncan commented 3 months ago

Environment

SaaS (https://sentry.io/)

Steps to Reproduce

When parsing CODEOWNER lines, we ignore lines which don't have an owner present.

For example (borrowed from GitHub), with the following lines:

# In this example, @octocat owns any file in the `/apps`
# directory in the root of your repository except for the `/apps/github`
# subdirectory, as its owners are left empty.
/apps/ @octocat
/apps/github # Skip

Expected Result

The line should exclude ownership of a subdirectory.

Actual Result

The line is ignored.

Product Area

Issues

Link

No response

DSN

No response

Version

No response

getsantry[bot] commented 3 months ago

Routing to @getsentry/product-owners-settings-projects for triage ⏲️

getsantry[bot] commented 3 months ago

Routing to @getsentry/product-owners-issues for triage ⏲️