Skip to content
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

Fix errors caused by force push #1927

Merged
merged 1 commit into from
Jun 11, 2017

Conversation

ethantkoenig
Copy link
Member

Fixes #1157

Do git fetch origin, followed by git reset --hard origin/<branch> in UpdateLocalCopyBranch(..) instead of a pull. This accounts for the case where others have made force pushes to origin, and the local repository maintained by gitea contains commits that are no longer in origin.

/cc @geek1011

@lunny lunny added this to the 1.2.0 milestone Jun 10, 2017
@andreynering
Copy link
Contributor

Trusted LGTM

@tboerger tboerger added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Jun 10, 2017
@appleboy appleboy added the type/enhancement An improvement of existing functionality label Jun 11, 2017
@appleboy
Copy link
Member

LGTM

@tboerger tboerger added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Jun 11, 2017
@appleboy appleboy merged commit e0a63a2 into go-gitea:master Jun 11, 2017
@ethantkoenig ethantkoenig deleted the fix/force_push branch June 12, 2017 13:42
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/enhancement An improvement of existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unusual bugs in web ui after force pushing
5 participants