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
refactor: ignore LGTM from author of pull request. #3283
Conversation
appleboy
commented
Jan 2, 2018
•
edited
edited
- ignore LGTM from the author of the pull request.
- remove MAINTAINERS file.
- remove ignore case in the pattern.
Codecov Report
@@ 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.
|
No, no. Why deleted the MAINTAINERS file? |
@lunny |
It will load team member from GitHub API but you can't restrict some member of Gitea team to use the |
@appleboy all members of team |
@lunny Yes, I know but I don't know why we need to keep the 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 |