Re-enable coverage

[techman83]

Renables coverage, any issues I'll debug and ensure my fork with open PR is up to date.

[techman83]

Double checked where I worked on this feature and it still works. I have changed to the 'demo' branch, but checking both the feat/demo branches are on the same commits

• Feat/environments CallyCo-io/Cally#29 (comment)
• https://github.com/techman83/coverage/commits/demo/workflow_run
• https://github.com/techman83/coverage/commits/feat/workflow_run

If something doesn't work, I'll fix it.

[HebaruSan]

If something doesn't work, I'll fix it.

Alright, brace for impact...!

[HebaruSan]

@techman83, should something have happened with this after merge? If so, then it looks like the first thing to fix will be getting it to run at all:

https://github.com/KSP-CKAN/NetKAN-Infra/actions/workflows/coverage-report.yml

... looks like coverage-build only runs when a PR is submitted, so I guess this is as-designed, and we'll have to wait for the next PR to test it?

NetKAN-Infra/.github/workflows/coverage-build.yml

Lines 3 to 6 in 8f6392c

	on:
	pull_request:
	branches:
	- master

[techman83]

That's expected. The workflow needs to exist in the default branch before it can run. This is to prevent someone skirting security by raising a PR that has a workflow that triggers a workflow within the repo, that would have extra permissions it wouldn't ordinarily get. It's why I had to write the changes in the first place.