Skip to content

Strip nil matches from code coverage output

Maxime requested to merge miouge/gitlab-ce:trim-nil-coverage into master

What does this MR do?

Strip nil matches from code coverage output.

Example regexp: (?:Coverage:\s(\d+)%)|(?:(\d+)% coverage) Results in:

  • Coverage: 100%: Not working, since the last match group has a nil result
  • 100% coverage: Working, since it is the last group match

What are the relevant issue numbers?

Does this MR meet the acceptance criteria?

Merge request reports