qiniu / reviewbot

Empower Your Code Quality with Self-Hosted Automated Analysis and Review
https://reviewbot-x.netlify.app
Apache License 2.0
35 stars 15 forks source link

feat: gitlab integration #428

Closed never112 closed 5 days ago

never112 commented 3 weeks ago
    • [x] 版本检查 无配置:>=10.8 版本 comments+discussion,<10.8版本使用comments, 有配置:>=10.8 版本 使用配置,<10.8版本使用comments. 全局无配置,repo无配置,->无配置 全局无配置,repo有配置 ,->repo配置 全局有配置,repo有配置,->repo配置
    • [x] linter结果汇总 comment输出。
    • [x] 单条linter结果 discussion 输出。
    • [x] linter结果汇总 表格形式
    • [ ] 拉取代码使用 accesstoken 方式 github/gitlab 统一使用 accesstokne的方式拉取代码 需要重新设计?
    • [x] 解决linter检查 错误
    • [x] gitlab 使用 accesstoken 拉取代码,不改动原有逻辑, 在gitlab handler 中实现

report: comment: Fail:Folding image Fail: Unfolding image pass: image

report:discussion image

comments-table

image
netlify[bot] commented 3 weeks ago

Deploy Preview for reviewbot-x canceled.

Name Link
Latest commit 3ff170ba675e2da309a40f5700285c590a4aa20e
Latest deploy log https://app.netlify.com/sites/reviewbot-x/deploys/67396b989f643d0008e16e88
codecov[bot] commented 3 weeks ago

Codecov Report

Attention: Patch coverage is 1.28388% with 692 lines in your changes missing coverage. Please review.

Project coverage is 25.61%. Comparing base (9aab280) to head (3ff170b). Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
internal/linters/providergitlab.go 0.00% 436 Missing :warning:
server.go 0.00% 209 Missing :warning:
internal/linters/hunk.go 0.00% 25 Missing :warning:
main.go 0.00% 19 Missing :warning:
config/config.go 75.00% 2 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #428 +/- ## ========================================== - Coverage 30.13% 25.61% -4.53% ========================================== Files 30 31 +1 Lines 3796 4467 +671 ========================================== Hits 1144 1144 - Misses 2515 3185 +670 - Partials 137 138 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

qiniu-x[bot] commented 5 days ago

[Git-flow] Hi @never112, There are some suggestions for your information:


Rebase suggestions

  • Following commits seems generated via git merge

Merge remote-tracking branch 'upstream/master' into zhou1025

Merge branch 'zhou1025' of github.com:never112/reviewbot into zhou1025

Merge branch 'zhou1025' of github.com:never112/reviewbot into zhou1025

Merge branch 'zhou1025' of github.com:never112/reviewbot into zhou1025

  • Following commits have duplicated messages

fix linter error

fix linter error

Merge branch 'zhou1025' of github.com:never112/reviewbot into zhou1025

Merge branch 'zhou1025' of github.com:never112/reviewbot into zhou1025

Merge branch 'zhou1025' of github.com:never112/reviewbot into zhou1025

fix main conflict

fix main conflict

fix main conflict

Which seems insignificant, recommend to use git rebase command to reorganize your PR.

For other git-flow instructions, recommend refer to these examples.

If you have any questions about this comment, feel free to raise an issue here: - **https://github.com/qiniu/reviewbot**