Skip to content

Add CODEOWNERS file in cookiecutter#49

Merged
llucax merged 3 commits intofrequenz-floss:mainfrom
llucax:codeowners
Jun 12, 2023
Merged

Add CODEOWNERS file in cookiecutter#49
llucax merged 3 commits intofrequenz-floss:mainfrom
llucax:codeowners

Conversation

@llucax
Copy link
Contributor

@llucax llucax commented Jun 9, 2023

We'll have more filters that need this.

  • Add a function to get a key from the JSON file
  • Emphasize that TODOs need to be FIXED
  • Add CODEOWNERS file

Fix #2.

llucax added 3 commits June 9, 2023 16:57
We'll have more filters that need this.

Signed-off-by: Leandro Lucarella <luca-frequenz@llucax.com>
Signed-off-by: Leandro Lucarella <luca-frequenz@llucax.com>
A new option is added to enter a default code owner, and if none is
entered, a (weak) heuristic will be used to chose a default one
depending on the type of repo.

Defaults will only be used if the github-org is `frequenz-floss`,
otherwise just a TODO is added.

Signed-off-by: Leandro Lucarella <luca-frequenz@llucax.com>
@llucax llucax added type:enhancement New feature or enhancement visitble to users part:cookiecutter Affects the generation of projects using cookiecutter labels Jun 9, 2023
@llucax llucax self-assigned this Jun 9, 2023
@llucax llucax requested review from a team and Marenz and removed request for a team June 9, 2023 14:59
@llucax llucax changed the title Add a function to get a key from the JSON file Add CODEOWNERS file in cookiecutter Jun 9, 2023
@llucax llucax enabled auto-merge June 12, 2023 11:59
@llucax llucax merged commit 6b99d22 into frequenz-floss:main Jun 12, 2023
@llucax llucax deleted the codeowners branch June 14, 2023 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

part:cookiecutter Affects the generation of projects using cookiecutter type:enhancement New feature or enhancement visitble to users

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add CODEOWNERS file

2 participants