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

When creating new repository fsck option should be enabled #5817

Merged
merged 2 commits into from Jan 24, 2019

Conversation

5 participants
@lafriks
Copy link
Member

lafriks commented Jan 23, 2019

Fixes #5810

@lafriks lafriks added this to the 1.8.0 milestone Jan 23, 2019

@codecov-io

This comment has been minimized.

Copy link

codecov-io commented Jan 23, 2019

Codecov Report

Merging #5817 into master will increase coverage by 0.04%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5817      +/-   ##
==========================================
+ Coverage   37.87%   37.92%   +0.04%     
==========================================
  Files         328      328              
  Lines       48255    48256       +1     
==========================================
+ Hits        18276    18299      +23     
+ Misses      27350    27328      -22     
  Partials     2629     2629
Impacted Files Coverage Δ
models/repo.go 43.84% <100%> (+0.03%) ⬆️
modules/indexer/repo.go 68.7% <0%> (+5.34%) ⬆️
models/repo_indexer.go 50% <0%> (+5.5%) ⬆️
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 3b36402...e43a55b. Read the comment docs.

@bkcsoft bkcsoft added the lgtm/need 1 label Jan 23, 2019

@lafriks lafriks force-pushed the lafriks:fix/fsck_enabled_by_default branch from beed3f6 to 429439c Jan 23, 2019

@lafriks lafriks force-pushed the lafriks:fix/fsck_enabled_by_default branch from 429439c to 6b58b6e Jan 23, 2019

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

@techknowlogick techknowlogick merged commit 386e531 into go-gitea:master Jan 24, 2019

2 checks passed

approvals/lgtm this commit looks good
continuous-integration/drone/pr the build was successful
Details
@techknowlogick

This comment has been minimized.

Copy link
Member

techknowlogick commented Jan 24, 2019

@lafriks please backport 😄

@lafriks lafriks deleted the lafriks:fix/fsck_enabled_by_default branch Jan 24, 2019

bmackinney added a commit to bmackinney/gitea that referenced this pull request Jan 27, 2019

lafriks added a commit to lafriks/gitea that referenced this pull request Jan 28, 2019

lunny added a commit that referenced this pull request Jan 29, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment