farmer icon indicating copy to clipboard operation
farmer copied to clipboard

Update PublishCodeCoverageResults to v2

Open bothzoli opened this issue 1 year ago • 3 comments

This PR closes #1153

The changes in this PR are as follows:

  • Update Azure DevOps task version

I have read the contributing guidelines and have completed the following:

  • [ ] Tested my code end-to-end against a live Azure subscription.
  • [ ] Updated the documentation in the docs folder for the affected changes.
  • [ ] Written unit tests against the modified code that I have made.
  • [ ] Updated the release notes with a new entry for this PR.
  • [ ] Checked the coding standards outlined in the contributions guide and ensured my code adheres to them.

If I haven't completed any of the tasks above, I include the reasons why here: Change in Azure DevOps task, no change in source code. Cannot test pipeline run locally.

bothzoli avatar Jan 06 '25 01:01 bothzoli

No more warnings regarding deprecated task version. Unfortunately however, the new version does not support the detailed reporting that was available with V1. See also this GitHub issue.

A workaround mentioned is the usage if this Azure DevOps task.

Let me know whether you'd prefer to: 1 - Keep V1 with the detailed report disregarding warning messages 👉 Close this PR without merging 2 - Use V2 with the limited reporting capabilities 👉 Merge this PR 3 - Use aforementioned ADO task to provide detailed coverage report 👉 Let me know, and I'd be happy to rework this PR 4 - Other 👉 Let me know, I'd be happy to help

bothzoli avatar Jan 06 '25 01:01 bothzoli

I don't want to lose the detailed reports, so I would prefer one of the other options. Being able to actually see what is covered is beneficial, whereas a simple number is a useless metric.

Making v2 remarkably worse than v1 is laughable and so many people have pointed this out that I'm surprised they haven't either made a v3 or just abandoned v2 and stopped with the deprecation warnings.

ninjarobot avatar Jan 19 '25 22:01 ninjarobot

Yeap, I agree completely. Would you prefer keeping the depreciation warning messages or the third party ADO task?

bothzoli avatar Jan 20 '25 03:01 bothzoli