Skip to content
This repository has been archived by the owner on Nov 10, 2022. It is now read-only.

Added badges to README #84

Closed
wants to merge 1 commit into from

Conversation

matt-ross16
Copy link

Closes #50

Implemented a few badges to reflect to status of enarx/bot

Implemented a few badges to reflect to status of enarx/bot
Comment on lines +1 to +2
![hourly](https://github.com/enarx/bot/workflows/hourly/badge.svg)
![cleanup-sprint-board](https://github.com/enarx/bot/workflows/cleanup-sprint-board/badge.svg)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • The hourly workflow will be removed soon-ish. While the badge makes sense now, it likely won't in a few weeks. I'm still fine with the change, though.

  • The cleanup-sprint-board workflow will remain, though it's only ever run manually at the end of every sprint, so it's not super helpful to have it. I'd personally drop this one.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will there be a new Workflow added, or is it best to just move away from these badges?

Comment on lines +3 to +4
[![Average time to resolve an issue](http://isitmaintained.com/badge/resolution/enarx/bot.svg)](http://isitmaintained.com/project/enarx/bot "Average time to resolve an issue")
[![Percentage of issues still open](http://isitmaintained.com/badge/open/enarx/bot.svg)](http://isitmaintained.com/project/enarx/bot "Percentage of issues still open")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not opposed to merging these, but it's worth noting that we haven't really been using issues in a high-throughput way like some of Enarx's other repos. They've mostly been used to denote larger tasks that get done in a few weeks, or even for backlogged tasks that have never been started. As such, the numbers on these badges look way worse than I believe they are.

Copy link
Contributor

@mbestavros mbestavros left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've left comments inline. Not opposed to merging this, but I think it has limited utility. As discussed in #50, we really need these badges on repos that use the Enarxbot instead of on Enarxbot directly.

Another alternative would be to go to the other extreme and put all badges for every workflow across the Enarx org in one place. That would allow us to see if any of them are failing without having to triage them individually.

@mbestavros
Copy link
Contributor

Closing this, as it's not quite what we want for #50.

@mbestavros mbestavros closed this Oct 19, 2020
Planning automation moved this from Triage to Done Oct 19, 2020
@puiterwijk puiterwijk removed this from Done in Planning Nov 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add status badges to README.md
4 participants