Skip to content

Use --force-with-lease instead of --force - #80

Merged
ZPascal merged 1 commit into
ad-m:masterfrom
imba-tjd:patch-1
May 4, 2022
Merged

ZPascal merged 1 commit into
ad-m:masterfrom
imba-tjd:patch-1

Conversation

@imba-tjd

@imba-tjd imba-tjd commented Jan 26, 2021 •

Copy link
Copy Markdown
Contributor

From my understanding, there are only benefits using --force-with-lease than --force. Correct me if I'm wrong.

@ZPascal

ZPascal commented May 4, 2022

Copy link
Copy Markdown
Collaborator

@imba-tjd I fully agree to use --force-with-lease instead of --force.

@ZPascal
ZPascal merged commit f43881e into ad-m:master May 4, 2022
@ZPascal

ZPascal commented May 6, 2022 •

Copy link
Copy Markdown
Collaborator

Hi @imba-tjd, I've reverted the change because it's results in a bunch of issues. I think it would make sense to have such a sensitive force push functionality. I've created a separate PR that includes the functionality in a smoother way.

@ZPascal ZPascal mentioned this pull request May 6, 2022
2 tasks done
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.

2 participants