Update Codeowners file #609
Conversation
Signed-off-by: bmorelli25 <[email protected]>
| /explore-analyze/ @elastic/platform-docs | ||
| /extend/ @elastic/platform-docs | ||
| /get-started/ @elastic/platform-docs | ||
| /manage-data/ @elastic/platform-docs |
There was a problem hiding this comment.
this one needs some nuance - /manage-data/ingestion/ needs to go to obs
| /solutions/ @elastic/obs-docs @elastic/search-docs @elastic/security-docs | ||
| /solutions/observability/ @elastic/obs-docs | ||
| /solutions/search/ @elastic/search-docs | ||
| /solutions/security @elastic/security-docs |
There was a problem hiding this comment.
not sure what the rules are but this one breaks the pattern
| /solutions/security @elastic/security-docs | |
| /solutions/security/ @elastic/security-docs |
| /extend/ @elastic/platform-docs | ||
| /get-started/ @elastic/platform-docs | ||
| /manage-data/ @elastic/platform-docs | ||
| /reference/ @elastic/platform-docs |
There was a problem hiding this comment.
we have some shared ownership down below here. there's obs, sec, ingest sections. don't know how picky we want to be about splitting up the folder, but platform shouldn't be the only owner
There was a problem hiding this comment.
Agreed, I think at least the following makes sense. For data-analysis and ingestion-tools they are split across the teams. data-analysis we could subfolder. Ingestion-tools I think maybe should be all obs-docs by the looks of it.
| folder | team |
|---|---|
| reference/data-analysis | ? |
| reference/elasticsearch/clients | platform-docs |
| reference/glossary | docs |
| reference/ingestion-tools | ? |
| reference/observability | obs-docs |
| reference/search | platform-docs |
| reference/security | security-docs |
| /manage-data/ @elastic/platform-docs | ||
| /reference/ @elastic/platform-docs | ||
| /release-notes/ @elastic/docs | ||
| /solutions/ @elastic/obs-docs @elastic/search-docs @elastic/security-docs |
There was a problem hiding this comment.
I'd skip using search docs (@georgewallace to confirm)
There was a problem hiding this comment.
Agree with Shaina, for now can just do platform-docs
| /release-notes/ @elastic/docs | ||
| /solutions/ @elastic/obs-docs @elastic/search-docs @elastic/security-docs | ||
| /solutions/observability/ @elastic/obs-docs | ||
| /solutions/search/ @elastic/search-docs |
| /solutions/observability/ @elastic/obs-docs | ||
| /solutions/search/ @elastic/search-docs | ||
| /solutions/security @elastic/security-docs | ||
| /troubleshoot/ @elastic/docs |
There was a problem hiding this comment.
we could be more targeted about the child items here. suggest we apply the same strategy to this section as we do to reference
There was a problem hiding this comment.
Agree., the sub folders make quite a bit of sense to me.
| /extend/ @elastic/platform-docs | ||
| /get-started/ @elastic/platform-docs | ||
| /manage-data/ @elastic/platform-docs | ||
| /reference/ @elastic/platform-docs |
There was a problem hiding this comment.
Agreed, I think at least the following makes sense. For data-analysis and ingestion-tools they are split across the teams. data-analysis we could subfolder. Ingestion-tools I think maybe should be all obs-docs by the looks of it.
| folder | team |
|---|---|
| reference/data-analysis | ? |
| reference/elasticsearch/clients | platform-docs |
| reference/glossary | docs |
| reference/ingestion-tools | ? |
| reference/observability | obs-docs |
| reference/search | platform-docs |
| reference/security | security-docs |
| /manage-data/ @elastic/platform-docs | ||
| /reference/ @elastic/platform-docs | ||
| /release-notes/ @elastic/docs | ||
| /solutions/ @elastic/obs-docs @elastic/search-docs @elastic/security-docs |
There was a problem hiding this comment.
Agree with Shaina, for now can just do platform-docs
| /solutions/observability/ @elastic/obs-docs | ||
| /solutions/search/ @elastic/search-docs | ||
| /solutions/security @elastic/security-docs | ||
| /troubleshoot/ @elastic/docs |
There was a problem hiding this comment.
Agree., the sub folders make quite a bit of sense to me.
| /explore-analyze/ @elastic/platform-docs | ||
| /extend/ @elastic/platform-docs | ||
| /get-started/ @elastic/platform-docs | ||
| /manage-data/ @elastic/platform-docs |
There was a problem hiding this comment.
| /manage-data/ @elastic/platform-docs | |
| /manage-data/ @elastic/platform-docs | |
| /manage-data/ingestion/ @elastic/obs-docs |
| /extend/ @elastic/platform-docs | ||
| /get-started/ @elastic/platform-docs | ||
| /manage-data/ @elastic/platform-docs | ||
| /reference/ @elastic/platform-docs |
There was a problem hiding this comment.
| /reference/ @elastic/platform-docs | |
| /reference/ @elastic/docs | |
| /reference/data-analysis/ @elastic/docs | |
| /reference/data-analysis/kibana/ @elastic/platform-docs | |
| /reference/data-analysis/machine-learning/ @elastic/platform-docs | |
| /reference/data-analysis/observability/ @elastic/obs-docs | |
| /reference/elasticsearch/clients/ @elastic/platform-docs | |
| /reference/glossary @elastic/docs | |
| /reference/ingestion-tools/ @elastic/obs-docs | |
| /reference/observability @elastic/obs-docs | |
| /reference/search @elastic/platform-docs | |
| /reference/security @elastic/security-docs |
| /manage-data/ @elastic/platform-docs | ||
| /reference/ @elastic/platform-docs | ||
| /release-notes/ @elastic/docs | ||
| /solutions/ @elastic/obs-docs @elastic/search-docs @elastic/security-docs |
There was a problem hiding this comment.
| /solutions/ @elastic/obs-docs @elastic/search-docs @elastic/security-docs | |
| /solutions/ @elastic/obs-docs @elastic/platform-docs @elastic/security-docs |
| /release-notes/ @elastic/docs | ||
| /solutions/ @elastic/obs-docs @elastic/search-docs @elastic/security-docs | ||
| /solutions/observability/ @elastic/obs-docs | ||
| /solutions/search/ @elastic/search-docs |
There was a problem hiding this comment.
| /solutions/search/ @elastic/search-docs | |
| /solutions/search/ @elastic/platform-docs |
Summary
This PR updates codeowners in this repository. I am looking for feedback. This is just a draft and I'm certain I got some things wrong. Thank you!