Quick access to merge status anywhere in pull requests [public preview] #188033
Replies: 35 comments 22 replies
-
|
Did it intend to remove "Update with rebase" option from PR show page? |
Beta Was this translation helpful? Give feedback.
-
|
We’ve seen the update branch via rebase option disappear since this was released. See related thread on this: https://github.com/orgs/community/discussions/188567 Can this be addressed? Thanks! |
Beta Was this translation helpful? Give feedback.
-
|
Hi @opti and @broksonic21, we appreciate you reaching out to report this issue! The GitHub team is aware of this and is working on a solution. We'll provide details when they're available in this discussion. Please subscribe to that post to receive real-time notifications on updates. If you have further questions or additional details about your experience, please comment directly there as well. |
Beta Was this translation helpful? Give feedback.
-
|
Love it! Dont need to scroll down everytime somethings change / breaks |
Beta Was this translation helpful? Give feedback.
-
|
This is great --one nit pick: I have a lot of vertical realestate but there is a small window which checks show up in, so I need to scroll to see all of them. I realize you're probably re-using the same component UI-wise, however, it would be much better if this component would eliminate the need to scroll so much in a small nested i-frame-like-thing to see all checks. |
Beta Was this translation helpful? Give feedback.
-
|
Did this change remove the Edit button that allows changing the base branch for a pull request? |
Beta Was this translation helpful? Give feedback.
-
|
Useful 👍🏽 |
Beta Was this translation helpful? Give feedback.
-
|
This change is visually awful. For the repositories I interact with, it just means that there is now a giant distracting red X at the top of every single PR, which conveys no useful information and is not actionable. Please bear in mind that some repositories use an alternative mechanism to approve merges, and this change hurts anyone involved with those repositories. |
Beta Was this translation helpful? Give feedback.
-
|
If there's room on the screen, I think the status of more checks should be displayed. It seems odd to require scrolling the in list of checks when there's a lot of whitespace below the merge button |
Beta Was this translation helpful? Give feedback.
-
|
Nice change. |
Beta Was this translation helpful? Give feedback.
-
|
Minor copy: When a PR is missing a review the language is forwardly geared to "Awaiting approval". It should perhaps be something neutral like "Awaiting review"? |
Beta Was this translation helpful? Give feedback.
-
|
Combining this view with the Overview info of the PR would be better |
Beta Was this translation helpful? Give feedback.
-
|
I like this change so far, thank you for working on this. One suggestion: as a reviewer I would like to be able to submit my review from this widget too. It feels natural that as it says |
Beta Was this translation helpful? Give feedback.
-
|
it's really nice |
Beta Was this translation helpful? Give feedback.
-
|
This change is nice, BUT would be much more helpful if the button was visible all the time when scrolling (within the banner containing PR's status, title etc). Right now you can jump into checks when you're either top of the page (button triggering side panel) or bottom (widget). When you're in the middle, you just have to jump top/bottom anyway, and lose the current context you're in. PS. it works like this in diff tab, but not in conversation tab. |
Beta Was this translation helpful? Give feedback.
-
|
The Edit button that is usually there isn't there anymore. I had to "switch to classic experience" I need that edit button to rebase the PR to a different base branch. |
Beta Was this translation helpful? Give feedback.
-
|
Like the change, small nit: Whenever you enter a PR page, the GitHub icon in the tab receives a status indicator according to the merge status (which makes it really useful to keep GitHub tabs open when waiting for some PRs). When opening and closing the sidebar via the new button, this indicator in the tab vanishes and only reappears when reloading the page. Not super vital, but a bit annoying nonetheless. |
Beta Was this translation helpful? Give feedback.
-
|
This is awesome!! Would it be possible to make it full height though because there's a bit of empty space at the bottom of the drawer where it would be nice if it would show more CI checks - see:
|
Beta Was this translation helpful? Give feedback.
-
Beta Was this translation helpful? Give feedback.
-
|
This is great! What would be nice is if it was also possible to approve a PR right within this quick access panel. |
Beta Was this translation helpful? Give feedback.
-
|
Please for the love of god let me approve PR's without having to switch to the Code panel. Especially in this day and age of AI... |
Beta Was this translation helpful? Give feedback.
-
|
Meh, it's okay I guess. It seems an incredibly wasteful use of screen real estate in most repositories and somehow very cluttered in others with large numbers of check results. I'd much rather see GitHub focus on your chronic availability issues. You have been offline for more than six hours every week of this year... |
Beta Was this translation helpful? Give feedback.
-
|
I really love this feature tbh, tho i would really love if we're able to see the information about an auto merge, right now do we have to disable the auto merge to check the commit title and description and then enable it again |
Beta Was this translation helpful? Give feedback.
-
|
This change will improve time I spend checking PRs now that we have agents commenting and the scroll is not longer to get to checks, this is magnific! |
Beta Was this translation helpful? Give feedback.
-
|
For those of us programmatically merging PRs, is there a way to get this information from the API? |
Beta Was this translation helpful? Give feedback.
-
|
Can we get more information when changes are requested? Like great I can see it up top, but with only the persons name and no reference to the comment makes it very useless. |
Beta Was this translation helpful? Give feedback.
-
|
Run auto fix and merge pull request to draft and rename it to Rakgoale Ai Developer Services |
Beta Was this translation helpful? Give feedback.
-
Beta Was this translation helpful? Give feedback.
-
|
Not sure if this has been mentioned but could you refactor the View checks button so that it opens a new tab? |
Beta Was this translation helpful? Give feedback.
-
|
this would be practical in GH mobile app as well. |
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.
-
We are rolling out the pull request merge status at the top of every pull request page! Check merge readiness from anywhere in the pull request experience, including the new "Files changed" page. Click to open the details so you can quickly identify blockers, missing approvals, and get your pull request ready to merge.
If you're still using the classic "Files changed" page, you'll need to opt into the new experience to view the merge status from this page. Learn more by checking out our changelog about the new experience.
Merge.status.button.mp4
🌟Leave a comment!
Join the discussion and leave feedback in the comments below!
Beta Was this translation helpful? Give feedback.
All reactions