Skip to content

Fix repo indexer not updating upon push#9957

Merged
lafriks merged 4 commits into
go-gitea:masterfrom
guillep2k:fix-indexer-reg
Jan 24, 2020
Merged

Fix repo indexer not updating upon push#9957
lafriks merged 4 commits into
go-gitea:masterfrom
guillep2k:fix-indexer-reg

Conversation

@guillep2k
Copy link
Copy Markdown
Member

This PR fixes a regression where the code indexer would not update after pushing to a repository.

May fix #9946

@lunny lunny added the type/bug label Jan 24, 2020
@lunny lunny added this to the 1.12.0 milestone Jan 24, 2020
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Jan 24, 2020
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #9957 into master will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #9957      +/-   ##
==========================================
+ Coverage   42.25%   42.28%   +0.02%     
==========================================
  Files         609      609              
  Lines       79692    79692              
==========================================
+ Hits        33673    33695      +22     
+ Misses      41866    41849      -17     
+ Partials     4153     4148       -5
Impacted Files Coverage Δ
services/pull/temp_repo.go 31.62% <0%> (-2.57%) ⬇️
models/repo.go 49.89% <0%> (+0.13%) ⬆️
models/error.go 31.13% <0%> (+0.36%) ⬆️
models/notification.go 65.4% <0%> (+1.26%) ⬆️
modules/queue/workerpool.go 43.77% <0%> (+2.57%) ⬆️
services/pull/patch.go 69.81% <0%> (+3.77%) ⬆️
modules/notification/ui/ui.go 80.85% <0%> (+6.38%) ⬆️

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 ea605aa...a7fe14e. Read the comment docs.

@GiteaBot GiteaBot 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 Jan 24, 2020
@lafriks lafriks merged commit dbd1fae into go-gitea:master Jan 24, 2020
@lafriks
Copy link
Copy Markdown
Member

lafriks commented Jan 24, 2020

Please send backport

@guillep2k guillep2k deleted the fix-indexer-reg branch January 24, 2020 19:17
@techknowlogick techknowlogick added the backport/done All backports for this PR have been created label Jan 25, 2020
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

backport/done All backports for this PR have been created 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.

Indexer returns no results for some terms

7 participants