Skip to content

test database is connect OK after db config initialized #239

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

Merged
merged 1 commit into from
Nov 24, 2016

Conversation

lunny
Copy link
Member

@lunny lunny commented Nov 24, 2016

No description provided.

@tboerger
Copy link
Member

Makes sense, LGTM

@tboerger tboerger added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. type/enhancement An improvement of existing functionality labels Nov 24, 2016
@tboerger tboerger added this to the 1.0.0 milestone Nov 24, 2016
@thibaultmeyer
Copy link
Contributor

LGTM

@tboerger tboerger added the lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. label Nov 24, 2016
@codecov-io
Copy link

codecov-io commented Nov 24, 2016

Current coverage is 3.03% (diff: 0.00%)

Merging #239 into master will decrease coverage by <.01%

@@            master      #239   diff @@
========================================
  Files           33        33          
  Lines         8106      8108     +2   
  Methods          0         0          
  Messages         0         0          
  Branches         0         0          
========================================
  Hits           246       246          
- Misses        7840      7842     +2   
  Partials        20        20          

Powered by Codecov. Last update b6b616b...0321050

@lunny lunny merged commit 450969c into go-gitea:master Nov 24, 2016
@lunny lunny removed the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Nov 24, 2016
@lunny lunny deleted the lunny/test_db_when_init branch November 24, 2016 14:30
@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