Skip to content

Comments

pick-to branch: improve guidance; restore branch on error/abort, etc.#101

Closed
DDvO wants to merge 5 commits intoopenssl:masterfrom
DDvO:pick-to-branch_guidance_fix-abort
Closed

pick-to branch: improve guidance; restore branch on error/abort, etc.#101
DDvO wants to merge 5 commits intoopenssl:masterfrom
DDvO:pick-to-branch_guidance_fix-abort

Conversation

@DDvO
Copy link
Contributor

@DDvO DDvO commented Nov 26, 2021

  • restore original commit HEAD of target on error/abort
  • improve user guidance and streamline interaction
  • allow 'h' or 'HEAD' as first argument, which leads to using the master HEAD
  • pick-to-branch: rename 'branch' to 'target' for clarity
  • restrict 'git pull --ff-only' to target branch

Copy link
Contributor

@paulidale paulidale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@DDvO
Copy link
Contributor Author

DDvO commented Dec 3, 2021

ready to merge

openssl-machine pushed a commit that referenced this pull request Dec 3, 2021
openssl-machine pushed a commit that referenced this pull request Dec 3, 2021
Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
(Merged from #101)
openssl-machine pushed a commit that referenced this pull request Dec 3, 2021
openssl-machine pushed a commit that referenced this pull request Dec 3, 2021
… using the master HEAD

Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Tomas Mraz <[email protected]>
(Merged from #101)
openssl-machine pushed a commit that referenced this pull request Dec 3, 2021
@t8m
Copy link
Member

t8m commented Dec 3, 2021

Merged. Thank you!

@t8m t8m closed this Dec 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants