-
Notifications
You must be signed in to change notification settings - Fork 20
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
actually error on internal non-allowed error #274
actually error on internal non-allowed error #274
Conversation
Converting to draft as I'm still getting a handle on what's best to do here |
I think it's better to go with #277 first and see how that goes |
Just to add a bit of context to what I think confused a lot of people: this kind of successful workflow run with errors being reported elsewhere is mostly an artifact of when TagBot was supposed to be run with a cron trigger, e.g. every hour or every day. GH's default email notifications send an email for every workflow failure, so a problem with your workflow file or some other recurring error would fill up your mailbox really quick (I think Chris R had hundreds of emails in a few hours when I released a bug one time, lol). |
I'm thinking it's time to just do this (or something similar). Gone are the days of cron. I think the errors will be more manageable and actionable, and perhaps even get more eyes on helping fix remaining issues. |
@christopher-dG I'm going to go ahead and merge and release this as 1.17.0 before the end of the week. Let me know if you disagree |
Fixes #261