Skip to content

Some tweaks for the Cobertura report#454

Open
oandrieu wants to merge 2 commits into
aantron:masterfrom
oandrieu:oandrieu/cobertura-ado-tweaks
Open

Some tweaks for the Cobertura report#454
oandrieu wants to merge 2 commits into
aantron:masterfrom
oandrieu:oandrieu/cobertura-ado-tweaks

Conversation

@oandrieu

Copy link
Copy Markdown

Hi, here are two small changes to the XML Cobertura report so that it integrates better with Azure DevOps.

  • use the OCaml module name rather than a file path for the class name attribute
  • add a command line option to specify the package name rather than the default .

Currently, the presence of paths in the name attribute results in the ADO report generator creating a single
enormous HTML file for all the instrumented source files ; the HTML report then cannot be displayed because
the HTML page is too large. Simply using the module name "fixes" the report generator so that it outputs one
HTML file per source file.

The package name is more of a minor thing : the . currently results in html files starting with . , so, hidden.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant