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

logging usability tweaks #102

Closed
beav opened this issue Aug 24, 2021 · 0 comments · Fixed by #107
Closed

logging usability tweaks #102

beav opened this issue Aug 24, 2021 · 0 comments · Fixed by #107

Comments

@beav
Copy link

beav commented Aug 24, 2021

Currently, a warning message is emitted when 'master' is not found on a repo, even though 'main' is tried next. The log level for this is too high and should be at 'info' level, since the user doesn't need to take action or look into anything if 'master' is not found.

Additionally, an info-level message for "trying alternate: main" is displayed, but there is no follow-up message saying if it worked or not. We can guess that it worked since bonfire continues processing but it would be good to be explicit that the branch was found, given the prior log messages.

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 a pull request may close this issue.

1 participant