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
feat(github): forkOrgs, forkCreate #18770
Conversation
commit a4832b2 Author: Rhys Arkins <rhys@arkins.net> Date: Fri Nov 4 14:50:22 2022 +0100 feat(github): forkOrgs, forkCreate
# Conflicts: # lib/modules/platform/github/index.spec.ts # lib/modules/platform/github/index.ts
My goals are to:
|
sounds good |
btw: if you fork to an org, the original repo maintainer can no longer update renovate branches in the fork by default. you would need to add them with write access to the fork to get it working. |
# Conflicts: # lib/modules/platform/github/index.spec.ts # lib/modules/platform/github/index.ts
The challenge is:
I'll still have a PAT on the backend (for doing the forking out of band) so maybe I could add a scheduled background process which detects maintainer in the parent repo and invites them to the forked repos. They may need to accept that invite though (which isn't necessary for maintainers can modify). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
otherwise LGTM
Co-authored-by: HonkingGoose <34918129+HonkingGoose@users.noreply.github.com>
Co-authored-by: HonkingGoose <34918129+HonkingGoose@users.noreply.github.com>
Co-authored-by: HonkingGoose <34918129+HonkingGoose@users.noreply.github.com>
Co-authored-by: HonkingGoose <34918129+HonkingGoose@users.noreply.github.com>
Merge queue setting changed
# Conflicts: # lib/modules/platform/github/index.ts
@zharinov can we detect a comment to force a test run? so we don't need to move out of draft mode? |
I don't think there is an easy way to do it. Seems like you either have permissions to re-run jobs, or you're not. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just a docs question, otherwise LGTM
🎉 This PR is included in version 35.132.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Changes
Adds new options forkOrgs and forkCreate
Context
These features will be used by me in the Forking Renovate app a part of some necessary migrations.
Documentation (please check one with an [x])
How I've tested my work (please tick one)
I have verified these changes via: