Branch protection rules prevent PRs that don't change tket or pytket from being merged

[cqc-alec]

The issue is the "build and test pytket (macos)" job, which is required, and should be skipped in such cases. For some reason, while the job is not run, the PR does not show it as skipped but as waiting for a runner (indefinitely), preventing merging.

For an example see this PR: #1114 .

[cqc-alec]

Required status checks:

[trvto]

This happens because some weird naming stuff when you use the matrix strategy. There is already a workaround implemented for the tket build. See job build_test_tket_not_required. If you add something similar for the pytket macos builds, should work.

[trvto]

Another possibility to fix this (which I personally find better) is to add a job at the end of the workflow file that fails if any other job fails. See https://github.com/CQCL/pytket-aqt/blob/develop/.github/workflows/tests.yml#L101 for an example. If you have such a job, you can configure github to only have it as a required check (for that workflow file). Which makes things a lot nicer I think