Skip to content

feat: migrate codeowners to urfave/cli#1313

Open
burmudar wants to merge 1 commit intomainfrom
wb/feat-codeowners
Open

feat: migrate codeowners to urfave/cli#1313
burmudar wants to merge 1 commit intomainfrom
wb/feat-codeowners

Conversation

@burmudar
Copy link
Copy Markdown
Contributor

@burmudar burmudar commented May 5, 2026

Migrates the codeowners subcommand to urfave/cli

Additional changes:

  • add a validator for flags which check that the value is empty, because even though a flag is set to Required: true you can still provie an empty value with -repo=
  • readFile now uses io.ReadAll and os.ReadFile

Test plan

Tested locally

@burmudar burmudar requested a review from a team May 5, 2026 13:32
@burmudar burmudar self-assigned this May 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant