-
Notifications
You must be signed in to change notification settings - Fork 54
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 results table #356
Conversation
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 good! Just a couple of minor points here and there.
src/main/resources/com/mathworks/ci/BuildArtifactAction/index.jelly
Outdated
Show resolved
Hide resolved
src/test/java/integ/com/mathworks/ci/BuildArtifactActionTest.java
Outdated
Show resolved
Hide resolved
src/test/java/integ/com/mathworks/ci/BuildArtifactActionTest.java
Outdated
Show resolved
Hide resolved
Maybe the blue ( #0074BA )for skip what we have been using for Github actions would be more consistent option. |
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
I've updated the MATLAB Build results table to be consistent with GitHub actions. Changes include:
|
The Green color somehow is not very prominent either we could consider making it bit bold or use bit darker shade |
* Added check for command textbox * Updated as per review comment * Updated as per review comment * Add build results support in Run Command step and skipped task logs link in table (#354) * add build table to command step and link to skipped logs * add skipped log link in status as well * fix pipeline failure * temp commit for checkout * update as per review comments * remove MatlabBuild.java and update tests * temp commit for checkout * update as per review * fix pipeline failure * format code * remove teardown method overriding * add super to method calls * Add skip reason for skipped tasks in build results table (#356) * Add skipReason in build results table * add skip reason texts * add test and bump Maven task * remove .idea from gitignore in this branch * update as per review comments * update as per review * update function name * update build results table * update test * update skip link * update status to succeeded * update data constructor --------- Co-authored-by: Nikhil Bhoski <47204011+nbhoski@users.noreply.github.com>
Fixes #344 by showing skip reason in build results table
Sample: