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
fix: show revision in badge when param is true #10545
Conversation
Signed-off-by: Thijs van Tol <43065692+thijsvtol@users.noreply.github.com>
Signed-off-by: Thijs van Tol <43065692+thijsvtol@users.noreply.github.com>
Signed-off-by: Thijs van Tol <43065692+thijsvtol@users.noreply.github.com>
Codecov ReportBase: 45.75% // Head: 45.75% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## master #10545 +/- ##
=======================================
Coverage 45.75% 45.75%
=======================================
Files 236 236
Lines 28527 28527
=======================================
Hits 13053 13053
Misses 13669 13669
Partials 1805 1805
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Signed-off-by: Thijs van Tol <43065692+thijsvtol@users.noreply.github.com>
@crenshaw-dev please review and merge |
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.
lgtm, thanks @thijsvtol!
Out of an abundance of caution, I'm only going to cherry-pick this into 2.5. If folks desperately need this in prior versions, I'll consult with #argo-contributors in Slack before cherry-picking.
* fix: show revision in badge when param is true Signed-off-by: Thijs van Tol <43065692+thijsvtol@users.noreply.github.com> * Update badge.go Signed-off-by: Thijs van Tol <43065692+thijsvtol@users.noreply.github.com> * Update badge.go Signed-off-by: Thijs van Tol <43065692+thijsvtol@users.noreply.github.com> * pr feedback Signed-off-by: Thijs van Tol <43065692+thijsvtol@users.noreply.github.com> Signed-off-by: Thijs van Tol <43065692+thijsvtol@users.noreply.github.com>
Cherry-picked onto release-2.5 for 2.5.0-rc2. |
Signed-off-by: Thijs van Tol 43065692+thijsvtol@users.noreply.github.com
Description:
If I use the api for retrieving the badges, the revision will always be shown when the
revision
query param is present.So for example if you call
https://sample.com/api/badge?name=tomato-service?revision=false
it will return the badge WITH the revision.So this PR fixes that it will only show the revision if the value is
true
Checklist: