Successful merge banner appears when a merge doesn't actually happen
See original GitHub issueDescription
Successful merge banner appears when clicking Update from master and it doesn’t take into account if an actual merge occurs.
Version
- GitHub Desktop: 1.5.1-beta0
- Operating system: 10.14.1 (18B75) - Mojave
Steps to Reproduce
- Find a branch that is up to date with master
- Click
update from master
from branch menu - Observe that the banner appears even though a merge didn’t occur
Expected Behavior
- The banner shouldn’t appear when a merge don’t actually happens
Actual Behavior
- The banner appears either way
Additional Information
Issue Analytics
- State:
- Created 5 years ago
- Reactions:1
- Comments:14 (14 by maintainers)
Top Results From Across the Web
Solved: Cannot merge pull request if any changes have been...
The work was done in different areas so there will be no merge conflicts when they are merged in. First pull request is...
Read more >How to use merge checks to enforce your master branch
Merge checks in Bitbucket make it easy to ensure that every Pull Request is fully vetted before it gets merged and keep your...
Read more >Troubleshooting Data Merge Errors - CreativePro Network
This can happen because there are not enough fields in the header row. To demonstrate this, take the following database. It has three...
Read more >GIT merge successfully but introduce a compilation error
Just fix it, commit and move on with your life. – Cristian. Aug 12, 2014 at 17:27. actually this happens ...
Read more >The Ethereum Merge is completed: Here's what's next
The Ethereum blockchain has successfully completed its shift away from proof-of-work to proof-of-stake (PoS) consensus following the merge ...
Read more >
Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free
Top Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Ooh I was thinking in the same banner area and it would disappear. Keeping the check would possibly still make sense as well.
This was fixed in https://github.com/desktop/desktop/pull/10453