Skip to content

Conversation

@lafriks
Copy link
Member

@lafriks lafriks commented Jun 20, 2017

If PR was made from forked repository and later fork was deleted Gitea was failing with panic. Fixed displaying commits and changed files tabs.

@lunny lunny added the type/bug label Jun 21, 2017
@lunny lunny added this to the 1.2.0 milestone Jun 21, 2017
@lunny
Copy link
Member

lunny commented Jun 22, 2017

LGTM

@tboerger tboerger added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jun 22, 2017
@pgaskin
Copy link
Contributor

pgaskin commented Jun 22, 2017

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 2 This PR needs two approvals by maintainers to be considered for merging. labels Jun 22, 2017
@bkcsoft
Copy link
Member

bkcsoft commented Jun 22, 2017

LGTM

@bkcsoft bkcsoft merged commit cc6b3f6 into go-gitea:master Jun 22, 2017
@lafriks lafriks deleted the fix_pr_display_deleted_fork branch June 30, 2017 07:35
@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/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants