search

open-sauced / pizza-cli

A CLI for all things OpenSauced
MIT License
31 stars 9 forks source link

fix: skip interactive steps in generate codeowners with --tty-disable flag #159

Closed nickytonline closed 1 week ago

nickytonline commented 1 week ago

Description

Now if the --tty-disable flag is passed to generate a codeowners, the interactive steps are skipped.

~There are currently no tests for codeowners.go. I'm going to look at adding some.~ (punting for now as mentioned below)

Related Tickets & Documents

Closes #154

Mobile & Desktop Screenshots/Recordings

CleanShot 2024-09-06 at 17 23 11

Steps to QA

  1. Run just build
  2. Run ./build/pizza generate codeowners --tty-disable
  3. Notice the command runs and skips the interactive steps.
  4. Run ./build/pizza generate codeowners
  5. Notice the command runs and does not skip the interactive steps.

Tier (staff will fill in)

[optional] What gif best describes this PR or how it makes you feel?