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.

Sign up for GitHub

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

Jump to bottom

Fork-originated Pull Requests not running github actions workflows #169

Closed
astrogeco opened this issue Jan 4, 2021 · 1 comment · Fixed by #171 or #172
Closed

Fork-originated Pull Requests not running github actions workflows #169

astrogeco opened this issue Jan 4, 2021 · 1 comment · Fixed by #171 or #172
Labels
bug continuous-integration
Milestone
Caelum

Comments

@astrogeco
Copy link
Contributor

Describe the bug

The current setup for the workflow runs on push instead of pull_request events so the tests are not running on pull requests from forks.

To Reproduce
Create a pull request from a fork.
See "checks" section for that pull request.
You will notice that the "Build and Test" workflows are not activated.

Expected behavior
"Build and Test" workflows are activated on pull requests and changes to them.

Code snips
If applicable, add references to the software.

Additional context
Add any other context about the problem here.
astrogeco added a commit to astrogeco/cFS that referenced this issue Jan 4, 2021
@astrogeco
Fix nasa#169, Trigger workflow on pull requests
b3ed26a 
Set fail-fast to false for all workflows
Clean up comments
@astrogeco astrogeco mentioned this issue Jan 4, 2021
Merged
astrogeco added a commit to astrogeco/cFS that referenced this issue Jan 4, 2021
@astrogeco
Fix nasa#169, Trigger workflow on pull requests
d1093f8 
Set fail-fast to false for all workflows
Clean up comments
astrogeco added a commit that referenced this issue Jan 5, 2021
@astrogeco
Merge pull request #171 from astrogeco/fix-167-and-169-cppcheck-and-p…
99d45e6 
…r-workflow

Fix #166 and #169, Add Static Code Analysis and Pull Request Trigger to CI workflow
@astrogeco astrogeco added this to the Caelum cFS-20.12 milestone Jan 5, 2021
@astrogeco astrogeco mentioned this issue Jan 6, 2021
Merged
@strieflin strieflin mentioned this issue Mar 10, 2022
Closed
chillfig pushed a commit to chillfig/cFS that referenced this issue Mar 17, 2022
@astrogeco
Merge pull request nasa#169 from jphickey/fix-164-chunk-test-size-not…
3b8e512 
…random

Fix nasa#164, consistent "chunk" test sizes
@angela-tran angela-tran mentioned this issue Jan 4, 2023
Merged
@gustavomm19
Copy link

gustavomm19 commented Jan 24, 2023 •
edited
Loading

how did you fixed this? I am having the same problem with my workflows: The current setup for the workflow runs on push instead of pull_request events so the tests are not running on pull requests from forks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug continuous-integration
Projects
None yet
Milestone
Caelum
2 participants
@gustavomm19 @astrogeco