Skip to content

Send full force flag to git push #127

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jan 28, 2014

Conversation

bsedat
Copy link
Contributor

@bsedat bsedat commented Jan 28, 2014

It seems like --f wasn't having the desired behavior for me (perhaps it was picking --force-with-lease?).

@robertodecurnex
Copy link
Contributor

-f
--force

    Usually, the command refuses to update a remote ref that is not an ancestor of the local ref used to overwrite it. This flag disables the check. This can cause the remote repository to lose commits; use it with care.

Has been working right for me.

@bsedat can you share you Git version and SO?

@bsedat
Copy link
Contributor Author

bsedat commented Jan 28, 2014

I'm using git version 1.8.5.3 on Mac OSX 10.9.1.
When I run a command like git push --f origin test:test in a shell I get the following error:
error: Ambiguous option: f (could be --force-with-lease or --follow-tags)

@robertodecurnex
Copy link
Contributor

Sorry, I was blind :P

It's -f. The second - shouldn't be there.

Your PR looks good. New I need to add some tests to prevent this errors in the furute.

@ghost ghost assigned robertodecurnex Jan 28, 2014
robertodecurnex added a commit that referenced this pull request Jan 28, 2014
Send full force flag to git push
@robertodecurnex robertodecurnex merged commit 935bded into ruby-git:master Jan 28, 2014
@bsedat
Copy link
Contributor Author

bsedat commented Jan 28, 2014

No worries at all! Thanks so much for the awesome gem.

@bsedat bsedat deleted the git-push-force branch January 29, 2014 02:47
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