Codecov Report

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

@@            Coverage Diff             @@
##           master    #3283      +/-   ##
==========================================
- Coverage   34.81%   34.81%   -0.01%     
==========================================
  Files         277      277              
  Lines       40263    40263              
==========================================
- Hits        14018    14016       -2     
- Misses      24184    24186       +2     
  Partials     2061     2061

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 274241c...3f452c2. Read the comment docs.

No, no. Why deleted the MAINTAINERS file?

@lunny ignore_maintainers_file set as true will ignore parse the maintainers file so we don't need this file anymore.

It will load team member from GitHub API but you can't restrict some member of Gitea team to use the L-GTM service.

@appleboy all members of team Maintainers on Gitea organization could L-GTM.

@lunny Yes, I know but I don't know why we need to keep the MAINTAINERS file? It always calls the GitHub API.

See: https://github.com/go-gitea/lgtm/blob/master/web/hook.go#L55-L74

@appleboy that's not for lgtm.

@lunny revered and change lgtm pattern to (?)LGTM.

@lunny @lafriks merge this PR?