-
Notifications
You must be signed in to change notification settings - Fork 43
Open
Labels
enhancementNew feature or requestNew feature or request
Description
What feature do you want to see added?
Currently, the status line that informs that the check is published looks like this:
[GitHub Checks] GitHub check (name: Jenkins, status: completed) has been published.
If the status is completed, it is also worth adding information about the conclusion (one of the values: action_required, cancelled, failure, neutral, success, skipped, stale, timed_out)
so log message will be as below:
[GitHub Checks] GitHub check (name: Jenkins, status: completed, conclusion: failure) has been published.
The code responsible for this:
github-checks-plugin/src/main/java/io/jenkins/plugins/checks/github/GitHubChecksPublisher.java
Line 99 in e9ce039
| buildLogger.log("GitHub check (name: %s, status: %s) has been published.", gitHubDetails.getName(), |
I have the impression that we just need to change the code as below, but I haven't checked that:
- buildLogger.log("GitHub check (name: %s, status: %s) has been published.", gitHubDetails.getName(),
- gitHubDetails.getStatus());
+ String conclusion = gitHubDetails.getConclusion();
+ if(conclusion != null) {
+ buildLogger.log("GitHub check (name: %s, status: %s, conclusion: %s) has been published.", gitHubDetails.getName(),
+ gitHubDetails.getStatus(), conclusion);
+ }else {
+ buildLogger.log("GitHub check (name: %s, status: %s) has been published.", +gitHubDetails.getName(),
+ gitHubDetails.getStatus());
+ }
+Upstream changes
No response
Are you interested in contributing this feature?
Unfortunately no, because it is too complicated from the point of view of corporate procedures.
beikov
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request