microsoft / mu_devops

Project Mu Developer Operations
https://microsoft.github.io/mu/
Other
26 stars 22 forks source link

Steps/UploadCodeCoverage.yml: Remove dependency and conditionalize [Rebase & FF] #274

Closed makubacki closed 10 months ago

makubacki commented 10 months ago

Contains two changes related to uploading code coverage:

  1. Remove edk2toollib dependency

    Some repos (like pure Rust repos) are expected not to depend on pytools. Since the codecov application can simply be run without RunCmd(), do that and prevent the need for those repos to depend on edk2toollib or unnecessary logic in the pipelines to bring it in.

  2. Conditionalize coverage upload steps on codecov token presence

    Some repos may not upload to codecov for various reasons. Those repos won't set the code coverage token so conditionalize the upload steps on the token.


Will unblock https://github.com/microsoft/mu_rust_hid/pull/12

makubacki commented 10 months ago

Merging. Tested in https://github.com/microsoft/mu_basecore/pull/625.