Skip to content

Improve tw:codeowners task

What does this MR do and why?

This improves the tw:codeowners rake task by grouping the entries for a directory if it is fully owned by the same writer. Also removes the leading . as I have read it has to be removed manually.

I think the way I solved this problem is probably not really the "Ruby Way of doing things" and probably also not the most efficient way so I'm happy for feedback in the review!

Screenshots or screen recordings

Example: Single file directories and jira were grouped without entries for each file

image

How to set up and validate locally

  1. bin/rake tw:codeowners > tmp/_tw
  2. View the tmp/_tw file

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

technical-writing#598 (closed) describes the process of using this task (as example the dot stripping)

Related issues

These issues and MRs describe the evolution of the CODEOWNERS project for the Technical Writing team, from the initial idea through multiple (!) iterations to a finished product. These links may have been added significantly after this issue or merge request was closed, so they may describe parts of the process before or after this work.

/cc @aqualls as you wrote the initial rake task

Related to Refine how to run batch updates for the CODEOWN... (technical-writing#668 - closed)

Edited by Amy Qualls

Merge request reports

Loading