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

Add docs about database migration #5661

Merged
merged 4 commits into from Jan 11, 2019

Conversation

7 participants
@Chouchen
Copy link
Contributor

Chouchen commented Jan 7, 2019

Update documentation for database migrations from Gogs as asked in #4163

Clement Desmidt
@codecov-io

This comment has been minimized.

Copy link

codecov-io commented Jan 7, 2019

Codecov Report

Merging #5661 into master will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5661      +/-   ##
==========================================
+ Coverage   37.75%   37.76%   +<.01%     
==========================================
  Files         323      323              
  Lines       47595    47595              
==========================================
+ Hits        17971    17975       +4     
+ Misses      27036    27031       -5     
- Partials     2588     2589       +1
Impacted Files Coverage Δ
models/repo_list.go 63.29% <0%> (-1.27%) ⬇️
routers/repo/view.go 46.01% <0%> (+1.22%) ⬆️
models/unit.go 14.28% <0%> (+14.28%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 31aa00f...f3e7de8. Read the comment docs.

@bkcsoft bkcsoft added the lgtm/need 2 label Jan 7, 2019

@lafriks

lafriks approved these changes Jan 8, 2019

@bkcsoft bkcsoft added lgtm/need 1 and removed lgtm/need 2 labels Jan 8, 2019

@lafriks lafriks added the kind/docs label Jan 8, 2019

@lunny
Copy link
Member

lunny left a comment

I think Launch every major version should before run Rewrite all update hook of repositories, that's not the correct sequence.

@bkcsoft bkcsoft added lgtm/done and removed lgtm/need 1 labels Jan 8, 2019

Clement Desmidt and others added some commits Jan 8, 2019

Clement Desmidt
@techknowlogick

This comment has been minimized.

Copy link
Member

techknowlogick commented Jan 10, 2019

@lunny changes have been made, please review.

@lunny

lunny approved these changes Jan 11, 2019

@techknowlogick techknowlogick merged commit 915278f into go-gitea:master Jan 11, 2019

2 checks passed

approvals/lgtm this commit looks good
continuous-integration/drone/pr the build was successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment