ci: disable scheduled action for forks

[JohelEGP]

See https://github.com/johelegp/units-1/actions/runs/604611174.

[mpusz]

Why do you want to disable this for forks? We need as many checks as possible in CI to confirm the quality of PRs.

If you are referring to the fact that CI broke than it is a temporary problem with Conan 1.34.0. I reported this issue to Conan developers and Conan 1.34.1 with the patch conan-io/conan#8568 is coming soon.

[JohelEGP]

Do all forks really need to run this on a schedule? Isn't it enough for the main repository to do it? I suspect most fork master branches stay at the commit they were forked from, which would leave them running them until they have 60 days of inactivity. The workflow should still run on PRs and pushes.

[mpusz]

I think I do not understand what do you mean by "on a schedule". I thought that this one is only run after new commits are pushed to the repo. If nothing is pushed this Action should not fire, right?

[JohelEGP]

It's also set to run on a schedule:
https://github.com/mpusz/units/blob/0d3740cc87cf1917df39b03a8a681d8965c481bd/.github/workflows/codeql-analysis.yml#L24-L25
See https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#onschedule.

[mpusz]

Ohh, my bad. I was not aware of this. It seems that I copy-pasted most of the action code and didn't notice that. I will remove the scheduled task ASAP.

[mpusz]

Closed with a22600c