-
Notifications
You must be signed in to change notification settings - Fork 38
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
Add the status checks property to allow publishing unstable builds as neutral checks #124
Conversation
Codecov Report
@@ Coverage Diff @@
## master #124 +/- ##
============================================
+ Coverage 82.37% 82.81% +0.43%
- Complexity 148 154 +6
============================================
Files 13 13
Lines 437 448 +11
Branches 41 41
============================================
+ Hits 360 371 +11
Misses 61 61
Partials 16 16
Continue to review full report at Codecov.
|
@@ -1,13 +1,7 @@ | |||
<?jelly escape-by-default='true'?> | |||
<j:jelly xmlns:j="jelly:core" xmlns:f="/lib/form"> | |||
<j:jelly xmlns:j="jelly:core" xmlns:f="/lib/form" xmlns:i="/io/jenkins/plugins/checks/github/status"> |
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.
this is the convention normally used
<j:jelly xmlns:j="jelly:core" xmlns:f="/lib/form" xmlns:i="/io/jenkins/plugins/checks/github/status"> | |
<j:jelly xmlns:j="jelly:core" xmlns:f="/lib/form" xmlns:g="/lib/github-checks/status"> |
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.
you mean I'll need to create new folders specifically to include these lib files?
Co-authored-by: Tim Jacomb <21194782+timja@users.noreply.github.com>
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.
I haven't tested manually but this looks sane, (not that I would ever use this feature)
context: jenkinsci/checks-api-plugin#46