Skip to content

Try to fix CONFLICT on pop#111

Open
ecarreras wants to merge 2 commits into
masterfrom
fix_pop_confict
Open

Try to fix CONFLICT on pop#111
ecarreras wants to merge 2 commits into
masterfrom
fix_pop_confict

Conversation

@ecarreras
Copy link
Copy Markdown
Member

@ecarreras ecarreras commented Nov 5, 2021

  • After doing git stash pop check if there are any unmerged file.
  • If there is an error reset the stash pop and keep the stash

We can undo the stash pop (keeping the stash) doing git reset --merge https://stackoverflow.com/questions/8515729/aborting-a-stash-pop-in-git

@ecarreras ecarreras self-assigned this Nov 5, 2021
@guilleJB
Copy link
Copy Markdown
Member

guilleJB commented Nov 9, 2021

Could generate a git diff file before to stash dirty code and delete it when all runs well?

@ecarreras
Copy link
Copy Markdown
Member Author

Could generate a git diff file before to stash dirty code and delete it when all runs well?

Why? the stash doesn't get deleted if we do a git reset --merge

@guilleJB
Copy link
Copy Markdown
Member

guilleJB commented Nov 9, 2021

Why? the stash doesn't get deleted if we do a git reset --merge

Sorry, I'm read git merge --reset not git reset --merge

@ecarreras ecarreras added the minor Minor auto version V_.X._ label Feb 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement minor Minor auto version V_.X._

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants