Allow PRs to be marked as merged by API or commit
#178103
Replies: 2 comments 1 reply
-
|
💬 Your Product Feedback Has Been Submitted 🎉 Thank you for taking the time to share your insights with us! Your feedback is invaluable as we build a better GitHub experience for all our users. Here's what you can expect moving forward ⏩
Where to look to see what's shipping 👀
What you can do in the meantime 💻
As a member of the GitHub community, your participation is essential. While we can't promise that every suggestion will be implemented, we want to emphasize that your feedback is instrumental in guiding our decisions and priorities. Thank you once again for your contribution to making GitHub even better! We're grateful for your ongoing support and collaboration in shaping the future of our platform. ⭐ |
Beta Was this translation helpful? Give feedback.
-
|
Local merges with GPG signing (especially squash or rebase) do not mark PRs as "Merged" on GitHub - they show as "Closed" because GitHub doesn't detect the PR head commit in the target branch. |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Select Topic Area
Product Feedback
Body
Considering of cli/cli#1318 and https://github.com/orgs/community/discussions/11639, if you want the merge commit of a PR is signed with your own GPG while using
merge with rebaseorsquash mergefeatures, it's necessary to do the merge local and then push the result to GitHub, instead of doing it on Web. cli/cli#1318 (comment) gives an example for this.However, when doing this, the PR will end up with
Closedinstead ofMerged, so is it possible to add an API endpoint and/or auto-scan commit message, so that PRs can be automatically marked asMergedwhen suchlocal mergesare pushed to GitHub?Currently, issues can be auto-closed with message
fix: #xx, perhaps we can allow a similar grammar that applies to PRs?Beta Was this translation helpful? Give feedback.
All reactions