anc95 / ChatGPT-CodeReview

🐥 A code review bot powered by ChatGPT
https://github.com/apps/cr-gpt
ISC License
3.77k stars 361 forks source link

装好action之后在提交代码的时候有检测但是没有提示结果 #59

Closed WXinFfly closed 1 year ago

WXinFfly commented 1 year ago

image image

anc95 commented 1 year ago

@WXinFfly 不是私有仓库的话,可以分享下链接?

WXinFfly commented 1 year ago

是私有仓库

WXinFfly commented 1 year ago

你需要什么信息,我可以告诉你 @anc95

anc95 commented 1 year ago

@WXinFfly 可能是下面两种情况跳过了 Review

  1. 文件的 diff 文本过长,超过 GPT 最大 token
  2. 文件是 rename 的或者 被 deleted 了
WXinFfly commented 1 year ago

@WXinFfly 可能是下面两种情况跳过了 Review

  1. 文件的 diff 文本过长,超过 GPT 最大 token
  2. 文件是 rename 的或者 被 deleted 了

image diff我试了也没有很长,我呢见没有rename货deleted yml文件是这样的: image 你帮我看一下是不是哪里配置有问题?

WXinFfly commented 1 year ago

@WXinFfly 可能是下面两种情况跳过了 Review

  1. 文件的 diff 文本过长,超过 GPT 最大 token
  2. 文件是 rename 的或者 被 deleted 了

image diff我试了也没有很长,我呢见没有rename货deleted yml文件是这样的: image 你帮我看一下是不是哪里配置有问题?

@anc95

anc95 commented 1 year ago

没看出什么问题 😭 @WXinFfly

WXinFfly commented 1 year ago

没看出什么问题 😭 @WXinFfly

@anc95 已经找到原因了,是因为没有指定gpt的Model

WXinFfly commented 1 year ago

@anc95 目前这个项目除了支持PR之外,是否支持正常的push?

anc95 commented 1 year ago

@WXinFfly synchronize 监听了这个事件就可以

WXinFfly commented 1 year ago

@anc95 我指的是直接push,不单独建立一个pullrequest的,是否可以监听?

anc95 commented 1 year ago

@WXinFfly 不支持的