-
Notifications
You must be signed in to change notification settings - Fork 451
Open
Description
Hello!
I'm using your app really really well.
Let me give you a suggestion.
Current Issue:
The code review conducted by the app includes deleted files, which is not necessary. In the code file src/bot.ts, the changedFiles
variable obtained from the data.data
object includes files with changes, including deleted files. The file.status
property can be one of added
, modified
, removed
, renamed
, or copied
. Therefore, it is suggested to exclude the files with the removed
and renamed
status from the changeFiles
. It may also be helpful to add an environment variable to control this behavior.
// src/bot.ts
let { files: changedFiles, commits } = data.data; // 68 line
Proposed Solution:
Exclude the files with the removed and renamed status from the changeFiles function, and consider adding an environment variable to control this behavior.
twilsonco
Metadata
Metadata
Assignees
Labels
No labels