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

Only run some workflows on the bevy repo (not forks) #9872

Merged
merged 1 commit into from
Sep 20, 2023

Conversation

CleanCut
Copy link
Member

@CleanCut CleanCut commented Sep 20, 2023

Objective

Eliminate unnecessary Actions CI builds on forks, such as:

  • Daily builds, which are a waste of compute on forks, even if they succeed (although the Android build fails)
  • Administrative builds that attempt to deploy something

In both the cases above, forks get CI failures that need to be ignored. It looks like this:

image image

Solution


…, not in forks -- things like deploys from the official repository or daily builds
@rparrett
Copy link
Contributor

Just chiming in with a thumbs-up. I was mega-confused by this earlier in some PR where "docs were failing."

@alice-i-cecile alice-i-cecile added the A-Build-System Related to build systems or continuous integration label Sep 20, 2023
@mockersf mockersf added this pull request to the merge queue Sep 20, 2023
Merged via the queue into bevyengine:main with commit dd7f800 Sep 20, 2023
23 checks passed
@CleanCut CleanCut deleted the official-workflows branch September 20, 2023 18:44
rdrpenguin04 pushed a commit to rdrpenguin04/bevy that referenced this pull request Jan 9, 2024
# Objective

Eliminate unnecessary Actions CI builds on forks, such as:
- Daily builds, which are a waste of compute on forks, even if they
succeed (although the Android build fails)
- Administrative builds that attempt to deploy something

In both the cases above, forks get CI failures that need to be ignored.
It looks like this:

<img width="1178" alt="image"
src="https://github.com/bevyengine/bevy/assets/5838512/6365059a-1170-4bba-9c60-3e252ae7779f">

<img width="1186" alt="image"
src="https://github.com/bevyengine/bevy/assets/5838512/ab824a0b-5202-42f7-a24f-95c5cd53376c">


## Solution

- [Only run some jobs when they are in the `bevyengine/bevy`
repo.](https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#example-only-run-job-for-specific-repository)
- Leave the rest of the workflows alone (you still get a full set of CI
for pull requests, for example)

---
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Build-System Related to build systems or continuous integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants