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 bug when merge pull request running on docker #6971

Closed
wants to merge 1 commit into from

Conversation

lunny
Copy link
Member

@lunny lunny commented May 17, 2019

Will fix #6946

@lunny lunny added the type/bug label May 17, 2019
@lunny lunny added this to the 1.9.0 milestone May 17, 2019
@lunny lunny mentioned this pull request May 17, 2019
7 tasks
@lafriks
Copy link
Member

lafriks commented May 17, 2019

Don't know why but test TestSearchRepo seems to fail

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label May 17, 2019
@zeripath
Copy link
Contributor

This is unnecessary since #6963 was merged and is included in #6961 which is also affected by the same bug in TestSearchRepo #6977

@lunny lunny closed this May 21, 2019
@lunny lunny deleted the lunny/fix_merge_pr_docker branch May 21, 2019 07:21
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error when merge pull request
4 participants