-
Notifications
You must be signed in to change notification settings - Fork 4
Add status badges to README.md #50
Comments
If you would like, I'm willing to implement some badges on your README |
@matt-ross16 Welcome! That would be awesome -- feel free to play around and submit a PR, if you'd like. The original intention of this issue was to broadcast whether the bot is currently failing or not. However, we're moving away from running our automation centralized on this repo, which may complicate things. A complete solution might involve putting status badges on all repos with automation enabled once we've made the necessary changes to run decentralized. Happy to chat about this, BTW -- if you're interested, drop by our chat and find the |
Thanks for the warm welcome! Which other repos will need connecting badges? Still trying to think of ones that would be beneficial to the project, so getting a glimpse of the other repos may help with my inspiration. |
@matt-ross16 Pretty much every repo that implements this bot's automation. (I anticipate that will include most, if not all, of Enarx's many repos.) For some background, the bot currently functions by running scripts on a scheduled hourly action on this repo. Each script (for example, something like This is all well and good, but running centralized has its drawbacks, and it's hard to set up. That makes it harder to generalize to other projects. Fortunately, some recent developments from Github have allowed for a newer, better design, which I've been working on for a while. In a nutshell, those same scripts that contain the organizational logic can now be consumed by other repos and run by their Actions infrastructure, instead of everything happening in one centralized place. This has few drawbacks, yet brings significant benefits for modularity and general reusability. The result of that work is likely finalizing this week, and we're going to transition to this new approach very shortly. That got long-winded, but that's basically a summary of what's happening with this repo! TL;DR, we'll need status badges in a lot of places :) |
Hey, sorry about my disappearance. Been trying to decide what the best way around this, but if you're moving away from your current structure, maybe I will wait it out and see what I can do once you are in the new setup, since it sounds like badges will be easier to manage in the new design scheme. |
@matt-ross16 Hey, no worries at all! I actually just opened the first PR for this migration here: enarx-archive/enarx-keepldr#84 This is actually pretty ideal for you -- that repo ( |
Thank you for referencing your recent pull request! I was able to mirror the status for the most part with my recent PR for the bot. |
Will the bot continue living, or is it likely going to become depreciated as the project decentralizes? Should I look into setting up similar badges across your repos? |
@matt-ross16 Thanks for the PR! I've left comments on it. I didn't mean to be unclear, but I meant to convey that the badging we want is actually on the repos that use the bot, and not the central |
Oh ok, I think I understand now. Time to start spreading my reach across the repos! So will any repo that uses the bot have an enarxbot.yml workflow? Is that the hint I should seek out? |
@matt-ross16 In a nutshell, yes! For now, the only workflow that does it is (BTW, are you on our chat instance? It might be better to have this conversation there instead of through Github. I've pinged you there.) |
I was just taking a look at the new badges that you added to the README, and I realized that they all were listed as "enarxbot" on the left of the badge, and it is not immediately obvious which workflow it is referring to (you have to hover over the button, and look at the displayed link). After doing some digging, I found that within the workflow file itself, a label can be made under https://github.com/marketplace/actions/bring-your-own-badge This will likely improve the utility of each badge on this README |
Might be a cool way to broadcast bot status.
The text was updated successfully, but these errors were encountered: