Skip to content

Conversation

ethantkoenig
Copy link
Member

@ethantkoenig ethantkoenig commented Aug 28, 2017

When a test that makes multiple requests fails, it's not always clear which request had the unexpected status code. This will hopefully make it a little easier to debug failing tests.

@lunny lunny added this to the 1.3.0 milestone Aug 28, 2017
@lafriks
Copy link
Member

lafriks commented Aug 28, 2017

LGTM

@tboerger tboerger added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Aug 28, 2017
@sapk
Copy link
Member

sapk commented Aug 28, 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 1 This PR needs approval from one additional maintainer to be merged. labels Aug 28, 2017
@lunny lunny merged commit d23b36c into go-gitea:master Aug 28, 2017
@ethantkoenig ethantkoenig deleted the test/error_msg branch September 10, 2017 00:19
@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/testing

5 participants