Allow PRs from forks to run tests? about plone.restapi

[wesleybl]

I see that we now have github actions in many Plone repositories. Should we add this in https://github.com/plone/meta?

CC @gforcada

[wesleybl]

@Akshat2Jain would you like to do a PR to fix this? It's basically putting pull_request in on in all .yml files here:

https://github.com/plone/plone.restapi/tree/master/.github/workflows

It is also necessary to put:

if: github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name != github.event.pull_request.base.repo.full_name

in all jobs. See:

plone/volto@5808b78

[Akshat2Jain]

@Akshat2Jain would you like to do a PR to fix this? It's basically putting pull_request in on in all .yml files here:

https://github.com/plone/plone.restapi/tree/master/.github/workflows

It is also necessary to put:

if: github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name != github.event.pull_request.base.repo.full_name

in all jobs. See:

plone/volto@5808b78

Yeah Sure!

if: github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name != github.event.pull_request.base.repo.full_name

So basically the above code states that this condition ensures that the job is only executed when the event is not a pull request or when the head repository (fork) is different from the base repository.

right? @wesleybl

[wesleybl]

@Akshat2Jain exactly! This prevents jobs from running twice if it's a PR made from the original repository.

[gforcada]

@wesleybl yes, contributions to plone/meta are always welcome, though plone.rest and plone.restapi are still not converted/adjusted to use plone/meta.

@tisto wanted to discuss that during the Bonn sprint, but did not happen 😅 but yeah, in a way or another it will be configured eventually, so contributions there to improve its quality and usability would be great 👍🏾