-
Notifications
You must be signed in to change notification settings - Fork 8.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[code-owners] Add area information to code owner entries #205143
Conversation
This adds a simple mapping mechanism between owners and areas of the Elastic organisation. Mappings have been defined statically in `packages/kbn-code-owners/src/code_owner_areas.ts`. The new area information has been added to `CodeOwnersEntry` objects. The `scripts/get_owners_for_file` script has been updated to display area information.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
💚 Build Succeeded
Metrics [docs]Public APIs missing comments
cc @dolaru |
Starting backport for target branches: 8.x |
) (cherry picked from commit fa1655e)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…) (#205147) # Backport This will backport the following commits from `main` to `8.x`: - [[code-owners] Add area information to code owner entries (#205143)](#205143) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"David Olaru","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-12-24T16:21:11Z","message":"[code-owners] Add area information to code owner entries (#205143)","sha":"fa1655e418db9fb5503d5b3ac88f57ffa7a7a061","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-minor","v8.18.0"],"title":"[code-owners] Add area information to code owner entries","number":205143,"url":"https://github.com/elastic/kibana/pull/205143","mergeCommit":{"message":"[code-owners] Add area information to code owner entries (#205143)","sha":"fa1655e418db9fb5503d5b3ac88f57ffa7a7a061"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/205143","number":205143,"mergeCommit":{"message":"[code-owners] Add area information to code owner entries (#205143)","sha":"fa1655e418db9fb5503d5b3ac88f57ffa7a7a061"}},{"branch":"8.x","label":"v8.18.0","branchLabelMappingKey":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: David Olaru <[email protected]>
Summary
This adds a simple mapping mechanism between owners and areas of the Elastic organisation.
Mappings have been defined statically in
packages/kbn-code-owners/src/code_owner_areas.ts
.The new area information has been added to
CodeOwnersEntry
objects.The
scripts/get_owners_for_file
script has been updated to display area information.