Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature: pizza generate config flow with existing config #174

Open
zeucapua opened this issue Sep 11, 2024 · 0 comments
Open

Feature: pizza generate config flow with existing config #174

zeucapua opened this issue Sep 11, 2024 · 0 comments

Comments

@zeucapua
Copy link
Contributor

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 #137 (review)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant