Ullrich Hafner

Results 174 comments of Ullrich Hafner

Sorry, seems that I forced pushed to your repository :-( Can you please force push your chagnes again? I wasn't aware that my branch is not set to track my...

Actually I just wanted to see the results in my Jenkins instance. From your code it also looks that HTML Is working in the description, doesn't it?

Currently, your headers are smaller than the text, maybe you need to tweak these a little bit.

BTW: in Java it might make more sense to use j2html to generate the HTML output (and not a `StringBuilder`). Example: https://github.com/jenkinsci/analysis-model/blob/565bacf88d429a00860e44cf4c8345acd97b5c92/src/main/java/edu/hm/hafner/analysis/parser/VeraCodePipelineScannerParser.java#L150

> The HTML tags I've tried seem to go through fine, but the CSS is stripped. For example, trying to add striping to the tables like this results in a...

> Another problem I noticed is that the suppression part of the Violation's specifics is getting lost somehow. It works in the ValgrindAdapter unit test, but Violation.getSpecifics().get("suppression") always returns null...

> > Do you have an example in the unit test? Then I can try to debug locally. > > Yes. 4 of the 5 test errors in the test...

I think we can postpone the problem with the parser and create a new release right now.

The new step `recordCoverage` is now in [beta testing](https://github.com/jenkinsci/code-coverage-api-plugin/releases/tag/v4.0.0), can you please check if this release helps to solve this issue? Please note that the syntax of the step is...

The new step `recordCoverage` is now in [beta testing](https://github.com/jenkinsci/code-coverage-api-plugin/releases/tag/v4.0.0), can you please check if this release helps to solve this issue? Please note that the syntax of the step is...