Skip to content

Conversation

@kevans91
Copy link
Contributor

This is a boolean flag; simply checking if it's set isn't enough, we must
check the value as well.

This is a boolean flag; simply checking if it's set isn't enough, we must
check the value as well.
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Apr 30, 2020
@techknowlogick techknowlogick added this to the 1.12.0 milestone Apr 30, 2020
@techknowlogick techknowlogick added the type/refactoring Existing code has been cleaned up. There should be no new functionality. label Apr 30, 2020
@GiteaBot GiteaBot 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 May 2, 2020
@guillep2k guillep2k added skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. and removed skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. labels May 2, 2020
@guillep2k
Copy link
Member

Ping LG-TM

@guillep2k guillep2k merged commit a5018d0 into go-gitea:master May 3, 2020
@kevans91 kevans91 deleted the skiprepobool branch May 3, 2020 04:04
ydelafollye pushed a commit to ydelafollye/gitea that referenced this pull request Jul 31, 2020
This is a boolean flag; simply checking if it's set isn't enough, we must check the value as well.

Co-authored-by: Lunny Xiao <[email protected]>
Co-authored-by: techknowlogick <[email protected]>
@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/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/refactoring Existing code has been cleaned up. There should be no new functionality.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants