-
Notifications
You must be signed in to change notification settings - Fork 39
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
Test Github Actions dashboard badge from meercode.io #1640
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1640 +/- ##
=======================================
Coverage 91.49% 91.49%
=======================================
Files 204 204
Lines 11175 11175
=======================================
Hits 10225 10225
Misses 950 950 Continue to review full report at Codecov.
|
Looks good. I got a badge that says "success rate 91%". |
Same here 👍 |
yeah cheers guys, but what do you get when you click on that badge? You should get a Dashboard with a lot of plots and info once you've logged in with yer GitHub credentials |
any luck with this, fellas? 🍺 |
Clicking the badge does not work that well for me. I first need to authorize the app to use my github account or I see nothing. After authorization, I do see white blobs representing the various test runs, but no plots or anything. |
Yes, same here: I think they're having some issues, as the Badge disappeared for me and the link of the badge (https://api.meercode.io/badge/ESMValGroup/ESMValCore?type=ci-success-rate&branch=main&lastDay=14) returns an |
I see this too, but clicking on the cards does nothing. Also, the fancy options on the left (e.g., summary) don't show anything. |
@valeriupredoi That's what it looks like for me too, but note that not even the branch names are filled in: it looks like all actual data is missing from the cards. |
OK cool! Progress! It seems they are indeed going through the bushes since I don't see any plots, data etc either - but normally one should be able to see that stuff, will check if they have some status report like GitHub - cheers for trying, guys! |
I sent some feedback to the devs, will get back to you guys when this starts working again - you fellas decided to test it right when it went belly up 🤣 |
OK plots are back for me, and I switched the dark mode on and they look super badass, can you fellas try again as well, please? I'll convert it to a RfR PR then 🍻 |
Yes, I can see the same now 🚀 |
cheers, Manu! Pretty nifty these plots aren't they? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine. I am a bit weary since it's not super tied to the project but just invites everyone to join meercode and there may be other projects sprinkled in, but why not try it out for a bit. I like the idea of having a dashboard that collates GH Actions and CircleCI in one place.
cheers much @zklaus 🍺
nay worry, I did the initial configuration for the ESMValGroup/ESMValCore being an admin of the repo, I don't think anyone can willingly add bits and bobs if they're not admins Circle is not yet added to the Meercode, do we want to add it? (not sure it's even possible tho) |
What I mean is that what you see on meercode.io is determined by your user account, not only by project configuration. But don't worry. I don't consider it a blocker. |
ah yeah fair enough - got it now. Meh, I guess that's fine for us few that will want to look at these things, pretty sure 85% of our users will not want to see/add all manners of plots anyway 😁 |
It's good to have CI Dashboard badges so we can have a bird's eye view of what our CI/CD is doing - @bouweandela pointed me to a dashboard from Circle, so I thought I'd have a go at a dash from Github Actions - proves out GA don't have a native dashboard so I found meercode.io offers a very decent integration (not only for GA) - can one of you @bouweandela @zklaus @schlunma please test if the badge is leading to the correct dashboard - it does for me, but I am registered with meercode, so not sure if that's working overall. Cheers 🍺
UPDATE at point of moving it to Ready for Review
Folks see plots and stats and all manners of dashboard-y stuff if they click on the Meercode badge, and autheticate with GitHub. Do we need documentation for this? I think we're fine like this -