Skip to content

Patch: Correct checkout hang and permission issues for forked PRs.#276

Open
mirsdemo wants to merge 3 commits into
Sleitnick:mainfrom
mirsdemo:patch-ci-logic
Open

Patch: Correct checkout hang and permission issues for forked PRs.#276
mirsdemo wants to merge 3 commits into
Sleitnick:mainfrom
mirsdemo:patch-ci-logic

Conversation

@mirsdemo
Copy link
Copy Markdown

@mirsdemo mirsdemo commented May 3, 2026

The previous workflow used github.head_ref, causing the CI to hang or fail on forked PRs, as it couldn't correctly resolve branch names across repositories. It also struggled with permission errors when trying to push back to contributors' forks.

This PR addresses and refactors the Generated Files workflow into a post-merge version cleaner model:

  • Switched to on: push -> main. This ensures the automation only runs once code has been merged.
  • By running directly on the base repository, we eliminate cross-repository permission errors, and the significant checkout hang.
  • Keeping this ensures that the README.md and filelist.json files are always in synchronization with the latest code on main, even if a contributor forgets to update them locally.

I verified on forks that the scripts executed successfully and that the bot can push the commit directly to the repository without authentication hurdles! :D

@IsraeliDaniel
Copy link
Copy Markdown

Any update? Waiting to be able to pass tests <3

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 this pull request may close these issues.

2 participants