Feature Req: Generate badges from JaCoCo reports in xml format about jacoco-badge-generator HOT 7 OPEN

I thought of an additional option, essentially a variation of the first one: Create a new input without csv/xml in its name, and deprecate the existing jacoco-csv-file input, but keep it around for now. It avoids the potential downside of just adding jacoco-xml-file (e.g., might cause confusion leading to false assumption that both reports are needed, as well as the possibility that jacoco might later introduce yet another format). But at the same time it temporarily avoids a breaking change.

from jacoco-badge-generator.

@japarson I'm not familiar with Pester so I took a look at it. Based on their docs it looks like only the xml report is generated. It looks like they adopted jacoco's xml format but not jacoco itself.

Anyway, I think xml report support is a reasonable request. It shouldn't be difficult to parse. I don't think an input to turn on option is likely necessary, and can probably just go by file extension. I know jacoco uses the standard xml extension when it generates an xml report. Can you confirm whether Pester's report uses xml file extension as well?

from jacoco-badge-generator.

@japarson I'm not familiar with Pester so I took a look at it. Based on their docs it looks like only the xml report is generated. It looks like they adopted jacoco's xml format but not jacoco itself.

Anyway, I think xml report support is a reasonable request. It shouldn't be difficult to parse. I don't think an input to turn on option is likely necessary, and can probably just go by file extension. I know jacoco uses the standard xml extension when it generates an xml report. Can you confirm whether Pester's report uses xml file extension as well?

Pester reports use the xml file extension by default:

Thank you so much for considering this request. I mentioned the new input option to turn it on because I saw there was an input specifically for csv files: jacoco-csv-file. Would I still use this input for an xml file?

from jacoco-badge-generator.

@japarson when I replied earlier I had forgotten that the input has csv in its name. So a new input definitely makes more sense.

from jacoco-badge-generator.

@japarson when I replied earlier I had forgotten that the input has csv in its name. So a new input definitely makes more sense.

I should have included my reasoning in the original post. My bad there.

Thinking it through some more, I think there are actually 2 options here:

1. Change the name of the existing input to jacoco-file and add some documentation about the possible input types.
   • This is a "major" level breaking change (in the context of semantic versioning).
2. Add a new input specifically for xml files jacoco-xml-file.
   • This is a "minor" level change.

I'm not sure what the best option is. What do you think?

from jacoco-badge-generator.

Hi @cicirello Thanks :). I wanted to know if this support is added or not? I am trying to use the command line utility but it doesn't work for xml with and without file type format jacoco-xml-file and jacoco-file.

from jacoco-badge-generator.

Hi @manoj-paypay no, it doesn't support JaCoCo XML reports yet, only the CSV reports at the present time.

from jacoco-badge-generator.