mathworks/jenkins-matlab-plugin

The build summary should not include an ordered list (1., 2.)

Closed this issue · 11 comments

mw-hrastega commented

Please replace the ordered list in the build summary (1. and 2. ) with an unordered list (bullet points). We should use an ordered list only if the order matters.

nbhoski commented

Fixed

Vahila commented

The bullet points are changed as shown below:
image

mw-hrastega commented

Hi, @nbhoski and @Vahila. After brainstorming with @sameagen-MW, I feel that we can improve the summary and be more consistent with other sections by streamlining the text and removing some of the whitespace. Can you please update the text and line spacing as follows? Thanks.

MATLAB Build Result

Tasks run: 3
Failed: 0
Skipped: 0

nbhoski commented

Fixed

mw-hrastega commented

Fixed

Thanks, @nbhoski . @Vahila, could you please verify the change and then attach a screenshot?

Vahila commented

The summary is updated as below
image

mw-hrastega commented

Thanks, @nbhoski and @Vahila . A few things to consider:

  • "Task run" should be "Tasks run" (missing s).
  • "Tasks run:", "Failed:", and "Skipped:" should be bolded.
  • The line spacing is quite large and makes the list elements disconnected.
nbhoski commented

Fixed
image

  • ❤️1
mw-hrastega commented

Fixed image

Nice job, Nikhil.

@sameagen-MW:

  1. Are the line spacings consistent with other sections of the Jenkins build summary?
  2. Are the numbers bold in the screenshot above?
  • 👍1
Vahila commented

The summary is updated as per suggestions.
image