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

Update build tags for sqlite_unlock notify #5144

Merged
merged 1 commit into from Oct 23, 2018

Conversation

5 participants
@typeless
Copy link
Contributor

typeless commented Oct 23, 2018

Fixes #2040

@lunny lunny added this to the 1.7.0 milestone Oct 23, 2018

@typeless typeless force-pushed the typeless:update-build-tags-for-unlock-notify branch from b896e2c to 74ac735 Oct 23, 2018

@codecov-io

This comment has been minimized.

Copy link

codecov-io commented Oct 23, 2018

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5144      +/-   ##
==========================================
- Coverage   37.49%   37.48%   -0.01%     
==========================================
  Files         309      309              
  Lines       45782    45782              
==========================================
- Hits        17164    17161       -3     
- Misses      26157    26159       +2     
- Partials     2461     2462       +1
Impacted Files Coverage Δ
models/repo_indexer.go 50.84% <0%> (-1.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 2ce72d4...b56c872. Read the comment docs.

@bkcsoft bkcsoft added the lgtm/need 2 label Oct 23, 2018

@typeless typeless force-pushed the typeless:update-build-tags-for-unlock-notify branch from 74ac735 to b56c872 Oct 23, 2018

@lunny

lunny approved these changes Oct 23, 2018

@bkcsoft bkcsoft added lgtm/need 1 and removed lgtm/need 2 labels Oct 23, 2018

@sapk

sapk approved these changes Oct 23, 2018

@bkcsoft bkcsoft added lgtm/done and removed lgtm/need 1 labels Oct 23, 2018

@lunny lunny merged commit 25c49cf into go-gitea:master Oct 23, 2018

2 checks passed

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

HoffmannP pushed a commit to HoffmannP/gitea that referenced this pull request Nov 14, 2018

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