ci: Fix code coverage command in YAML configuration

[unity-nikos]

Purpose of this PR

[CI] Fix code coverage command in YAML configuration

Coverage reporting was not setup due to this

Initiative was triggered from this Discussions thread

Jira ticket

n/a

Changelog

• Fixed: [CI] Fixed code coverage command in YAML configuration

Documentation

• Please follow the Codecov guide for packages, if you haven't

Notes to reviewers

• Code Coverage - NGO [ubuntu, 6000.3] should run on PR trigger AND on pushing changes to master/develop/develop-2.0.0
• It will need to be ported to develop branch, I guess

Testing & QA

• Code Coverage - NGO [ubuntu, 6000.3]

• com.unity.netcode.gameobjects in Codecov

Automated tests:

• Covered by existing automated tests
• Covered by new automated tests

Does the change require QA team to:

• Review automated tests?
• Execute manual tests?
• Provide feedback about the PR?

If any boxes above are checked the QA team will be automatically added as a PR reviewer.

Backports

Needs to be ported to develop branch potentially

[cla-assistant-unity]

All committers have signed the CLA.

[michalChrobot]

Thanks for catching this typo, I remember at some point I wanted to investigate why NGO doesn't show in CodeCov but I never got to it.

I wouldn't bother backporting to develop as we don't really backport anything there (develop branch corresponds to NGOv1.X version while develop-2.0.0 corresponds to lates NGOv2.X version on which we continue our development)

Code Coverage - NGO [ubuntu, 6000.3] should run on PR trigger

I think I would simply add it to daily trigger to not increase PR waiting time (as code coverage results wouldn't block the PR anyway and those tests are run itself by other jobs).

[unity-nikos]

com.unity.netcode.gameobjects in Codecov

[unity-nikos]

I think I would simply add it to daily trigger to not increase PR waiting time (as code coverage results wouldn't block the PR anyway and those tests are run itself by other jobs).

Please also add it as dependency to the PR Trigger job i.e. Minimal PR checks in pr_minimal_required_checks

In case you'd like to make use of the PR comment

Execution time of the coverage job is ~30min

Codecov check is informational: true, and it doesn't block PRs

[NoelStephensUnity]

Awesome!
Thank you for fixing this!