Skip to content
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

Merged
merged 2 commits into from
Dec 24, 2024

Conversation

dolaru
Copy link
Member

@dolaru dolaru commented Dec 24, 2024

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.

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.
@dolaru dolaru added release_note:skip Skip the PR/issue when compiling release notes v9.0.0 backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) v8.18.0 labels Dec 24, 2024
@dolaru dolaru self-assigned this Dec 24, 2024
@dolaru dolaru requested a review from a team as a code owner December 24, 2024 14:21
Copy link
Contributor

@fake-haris fake-haris left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/code-owners 5 11 +6
Unknown metric groups

API count

id before after diff
@kbn/code-owners 12 22 +10

cc @dolaru

@dolaru dolaru merged commit fa1655e into elastic:main Dec 24, 2024
9 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/12484017389

@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Dec 24, 2024
…) (#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]>
stratoula pushed a commit to stratoula/kibana that referenced this pull request Jan 2, 2025
benakansara pushed a commit to benakansara/kibana that referenced this pull request Jan 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) release_note:skip Skip the PR/issue when compiling release notes v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants