anc95 / anc95/ChatGPT-CodeReview
How about excluding removed and renamed 'files' from code reviews?
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 4.5k
- Forks
- 461
- PR merge metrics
- No merged PRs in 30d
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.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start in src/bot.ts around the changedFiles assignment from data.data and inspect how each file's status is processed for review. Confirm the handling of removed and renamed files, then check the surrounding configuration conventions before deciding whether an environment variable is appropriate. Done means those statuses are excluded from reviews and the behavior is covered by the project's existing checks.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- github, typescript
- Domain
- devtools
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100