open-sauced / pizza-cli

A CLI for all things OpenSauced
MIT License
39 stars 11 forks source link

Feature: `pizza generate config` flow with existing config #174

Open zeucapua opened 1 month ago

zeucapua commented 1 month ago

Suggested solution

When the user already has a .sauced.yaml configuration file, the pizza generate config command should warn the user that it exists and give them the option of either regenerating/replacing it or use the existing users when attributing new emails.

Context

Having an existing .sauced.yaml, it regenerated it, but the missing premable in comments got removed. Not sure if that's from our template, but just mentioning it.

When regenerating the .sauced.yaml, if one is detected (non-interactive mode) potentially mention it before they proceed.

Also, not sure how far back we're looking by default in the git history, but I see Anush, one of our interns in the list.

CleanShot 2024-09-10 at 13 50 41

Also, we should skip known bot accounts, i.e. any account ending in [bot].

Originally posted by @nickytonline in https://github.com/open-sauced/pizza-cli/pull/137#pullrequestreview-2293271309