-
Notifications
You must be signed in to change notification settings - Fork 6
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 skip reason for skipped tasks in build summary table #49
Conversation
I've updated the MATLAB Build summary table to be consistent with Jenkins. Changes include:
|
src/buildSummary.ts
Outdated
} else { | ||
return [t.name, '🟢 Success', t.description, t.duration.toString()]; | ||
return [t.name, '🟢 Successful', t.description, t.duration]; |
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.
return [t.name, '🟢 Successful', t.description, t.duration]; | |
return [t.name, '🟢 Succeeded', t.description, t.duration]; |
The group preferred "Succeeded" over "Successful" to better match "Failed" and "Skipped".
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.
Thanks Mark for checking with the group! I preferred "Successful" more but I guess it's fair enough to use "Succeeded" as we have "Failed" and "Skipped"
Fixes #43 by showing skip reason in build summary table
Sample: