Skip to content

Conversation

lunny
Copy link
Member

@lunny lunny commented Apr 5, 2017

This will update xorm to a5cb21c@go-xorm/xorm and fix #1443

@lunny lunny added the type/bug label Apr 5, 2017
@lunny lunny added this to the 1.2.0 milestone Apr 5, 2017
@strk
Copy link
Member

strk commented Apr 5, 2017

LGTM, but why is cryptocast5 being also moved ? different govendor version ?

@tboerger tboerger added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Apr 5, 2017
@lunny
Copy link
Member Author

lunny commented Apr 6, 2017

@strk maybe it's govendor's bug?

@appleboy
Copy link
Member

appleboy commented Apr 6, 2017

LGTM

@appleboy
Copy link
Member

appleboy commented Apr 6, 2017

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 6, 2017
@lunny lunny merged commit 5acfc7c into go-gitea:master Apr 7, 2017
@lunny lunny deleted the lunny/fix_mssql_migrate_and_500 branch April 19, 2017 05:51
@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.

Organization Dashboard Error On MSSQL
4 participants