Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make workflow work when PR is opened from fork #19

Closed
efb4f5ff-1298-471a-8973-3d47447115dc opened this issue Oct 15, 2023 · 7 comments
Closed

Comments

@efb4f5ff-1298-471a-8973-3d47447115dc

When u create a PR from a fork the workflow will not run.
Please make it possible for the workflow to run when a pr is created from a fork

@gregsdennis
Copy link
Owner

Can you elaborate on "will not run"? What do you mean by this? Does it run but fail, or does it just not run at all?

@efb4f5ff-1298-471a-8973-3d47447115dc

it does not run at all

@gregsdennis
Copy link
Owner

@efb4f5ff-1298-471a-8973-3d47447115dc

See #20

@efb4f5ff-1298-471a-8973-3d47447115dc

I think that's a repository/fork setting, and not something to do with this action.

Please see

* https://docs.github.com/en/actions/managing-workflow-runs/approving-workflow-runs-from-public-forks

* https://github.blog/2020-08-03-github-actions-improvements-for-fork-and-pull-request-workflows/

This is actually not true we have workflows running in our repo that work fine when a PR is opened from fork
See https://github.com/FreeTubeApp/FreeTube/blob/development/.github/workflows/conflicts.yml

@efb4f5ff-1298-471a-8973-3d47447115dc

Its working -.- Welp apologies for taking your time i clearly misconfigured it somehow

@gregsdennis
Copy link
Owner

we have workflows running in our repo that work fine when a PR is opened from fork

I've noticed in other repos that some workflows run automatically from forks while others don't. I'm not sure what the criteria is for that to occur. I can explore that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants