Skip to content

support health check via / and fix #969 - #1520

Merged
lunny merged 1 commit into
go-gitea:masterfrom
lunny:lunny/health_check
Apr 20, 2017
Merged

support health check via / and fix #969#1520
lunny merged 1 commit into
go-gitea:masterfrom
lunny:lunny/health_check

Conversation

@lunny

@lunny lunny commented Apr 20, 2017

Copy link
Copy Markdown
Member

No description provided.

@lunny lunny added the type/enhancement An improvement of existing functionality label Apr 20, 2017
@lunny lunny added this to the 1.2.0 milestone Apr 20, 2017
@lunny

lunny commented Apr 20, 2017

Copy link
Copy Markdown
Member Author

will fix #969

@tboerger tboerger added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Apr 20, 2017
@appleboy

Copy link
Copy Markdown
Member

LGTM

@appleboy

Copy link
Copy Markdown
Member

let L-G-T-M work

@tboerger tboerger added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Apr 20, 2017
@strk

strk commented Apr 20, 2017

Copy link
Copy Markdown
Member

LGTM (and sounds like an easy thing to put under unit testing...)

@lunny

lunny commented Apr 20, 2017

Copy link
Copy Markdown
Member Author

let L-G-T-M work

@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 Apr 20, 2017
@lunny
lunny merged commit 4207278 into go-gitea:master Apr 20, 2017
@lunny
lunny deleted the lunny/health_check branch April 20, 2017 12:14
@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.

4 participants