-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
fix(gitlab): pr creator missing default for target_project_id #8985
Conversation
Fixes #8846 |
e9148e0
to
c85405a
Compare
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.
Thanks for the fix!
Don't worry about the composer test failures. The PPA we use appears to be down. |
c85405a
to
ea500ad
Compare
Can we add a test for this? |
@JamieMagee I just rebased and pushed again. Since I figured the PPA would be back online again by now. |
Unfortunately I don't have merge rights. Let me ping someone who does. |
Before #8729
target_project_id
wasnil
when it was not provided.Version ^0.240.0 upward is therefore broken for consumers on Gitlab.
This should be the default behaviour again.