Skip to content

Conversation

@BYK
Copy link
Member

@BYK BYK commented Dec 15, 2020

Based on https://docs.github.com/en/free-pro-team@latest/github/creating-cloning-and-archiving-repositories/about-code-owners#example-of-a-codeowners-file, the order matters and the last line that matches takes precedence. That makes all definitions above the * catch-all definition useless. This PR fixes that.

#skip-changelog

Based on https://docs.github.com/en/free-pro-team@latest/github/creating-cloning-and-archiving-repositories/about-code-owners#example-of-a-codeowners-file, the order matters and **the last line that matches** takes precedense. That makes all definitions above the `*` catch-all definition useless. This PR fixes that.
@BYK BYK requested review from a team, chadwhitacre and jan-auer December 15, 2020 11:23
@BYK BYK merged commit 85b0be7 into master Dec 15, 2020
@BYK BYK deleted the byk/meta/fix-codeowners branch December 15, 2020 11:47
jan-auer added a commit that referenced this pull request Dec 18, 2020
* master:
  feat: Enable use of multiple value types per field (#882)
  fix(user-report): Make all fields but event-id optional (#886)
  release: 20.12.1
  release: 20.12.0
  ci(release): Move to getsentry/publish for releases (#885)
  meta: Fix CODEOWNERS (#884)
  fix: Also build rdkafka with ssl if ssl feature is enabled (#881)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants