Skip to content

Conversation

@mrnugget
Copy link
Contributor

@mrnugget mrnugget commented Oct 16, 2020

Yesterday I noticed that when executing steps fails in a repository we simply show "No changes" in the status bars. If you execute steps in a lot of repositories that means you'll need to wait until all of them are finished to get feedback on whether one of them failed.

This changes the behaviour from this

before

to this:

screenshot_2020-10-16_16 28 24

I'm not super happy with the StatusText() and SingleLineError() methods and how to best interface them, but then again I never found wrapped errors easy to manage. If you have any ideas: let me know!

@mrnugget mrnugget marked this pull request as ready for review October 16, 2020 14:30
@mrnugget mrnugget requested a review from a team October 16, 2020 14:30
@eseliger
Copy link
Member

I'm not super happy with the StatusText() and SingleLineError() methods and how to best interface them, but then again I never found wrapped errors easy to manage. If you have any ideas: let me know!

No real idea off the top of my head but I also didn't find it too strange.

@mrnugget
Copy link
Contributor Author

Since there are no opposing voices, let's merge it then and see what happens.

@mrnugget mrnugget merged commit 729e9a1 into main Oct 19, 2020
@mrnugget mrnugget deleted the mrn/report-errors-as-errors branch October 19, 2020 07:36
@LawnGnome
Copy link
Contributor

Late to the party, but I think it's fine too. 🥳

scjohns pushed a commit that referenced this pull request Apr 24, 2023
* Show failed tasks as errors in statusbars

* Change formatting of status bar error

* Add changelog
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 this pull request may close these issues.

4 participants