-
Notifications
You must be signed in to change notification settings - Fork 696
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: Add allow_forking to github_repository #1649
base: main
Are you sure you want to change the base?
Conversation
a5857fe
to
793acb5
Compare
Thank you @pascal-hofmann! Do you mind helping me run these tests? They're passing for me on the main branch but failing on this branch with a Does it pass for you? |
@kfcampbell No, it does not. :( No idea what's wrong here… |
Any chance of getting this merged? |
is it forgotten? |
793acb5
to
9756290
Compare
I've just rebased this PR and resolved the conflicts. |
Is there anything it can be done to push this forward? In highly automated environments as ours, and trying to push innersourcing internally, this is a bump on the road. Let us know if there's something we can do to help you on this. Thanks! 🙏 |
Hey, what is blocking this PR? We'd love to have it. |
Resolves #1071
Additional info
Pull request checklist
Does this introduce a breaking change?
Please see our docs on breaking changes to help!
Type: Breaking change
label)false
(this is also the default stated in the REST API docs.Pull request type
Type: Feature