ci: skip x16-docs PDF fetch on fork PRs so the CI build action no longer fails on that#389
Open
irmen wants to merge 1 commit into
Open
ci: skip x16-docs PDF fetch on fork PRs so the CI build action no longer fails on that#389irmen wants to merge 1 commit into
irmen wants to merge 1 commit into
Conversation
The "Fetch x16-docs PDFs" step uses gh run download to pull artifacts from the X16Community/x16-docs repo. On pull requests from forks, the default GITHUB_TOKEN has restricted read-only permissions and cannot access other repositories, causing the build to fail. Add a condition to skip this step when the PR originates from a fork.
This was referenced May 19, 2026
irmen
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The "Fetch x16-docs PDFs" step uses gh run download to pull artifacts from the X16Community/x16-docs repo. On pull requests from forks, the default GITHUB_TOKEN has restricted read-only permissions and cannot access other repositories, causing the build to fail. Add a condition to skip this step when the PR originates from a fork.
You can see below 2 of my PR's that now fail their CI build checks because of this issue.
There is likely anotherr problem as well though, if the pdf artifact has been removed altogether from the x16-docs repo because it got stale/too old and github deemed it necessary to clean it....which seems to have happened as well. I don't know how to solve this? Re-trigger a x16-docs build to make a fresh build artifact for a while until it expires again? IS there a way to make github store the build artifacts longer?
