Skip to content
This repository has been archived by the owner on Mar 6, 2024. It is now read-only.

limit the number of files that can be reviewed by this tool #27

Merged
merged 1 commit into from
Mar 14, 2023

Conversation

harjotgill
Copy link
Contributor

@harjotgill harjotgill commented Mar 14, 2023

Summary by OpenAI

Purpose: Limit the number of files that can be reviewed by this tool.

  • New Feature: Added a new input option max_files to specify the maximum number of files that can be reviewed.
  • Bug fix: Added checks to ensure that the number of files being reviewed does not exceed this limit.
  • Refactor: Updated the code to improve readability and maintainability.

@github-actions
Copy link
Contributor

🤖 OpenAI

Thank you for letting me participate in this code review.

The purpose of this pull request is to limit the number of files that can be reviewed by this tool. The developer has added a new input option max_files to specify the maximum number of files that can be reviewed, and has also added checks to ensure that the number of files being reviewed does not exceed this limit. Additionally, they have made some other changes to the code to improve readability and maintainability.

Overall, these changes seem like useful additions to the codebase. However, I would suggest that the developer consider adding some validation or error handling around the commenter.comment() function calls, to ensure that they do not cause any unexpected behavior or errors. Additionally, it might be helpful to provide some documentation or examples on how to use the new comment_tag constant.

Here's the updated summary table:

Filename Summary
src/bot.ts Added logging to record the response time of OpenAI messages.
src/main.ts Added an input option to specify the maximum number of files that can be reviewed.
src/options.ts Added a new input option max_files to specify the maximum number of files that can be reviewed.
src/review.ts Added a new constant comment_tag to store an auto-generated comment tag. Added a check to ensure that the number of files being reviewed does not exceed the maximum allowed by the tool. Updated the commenter.comment() function calls to use the comment_tag constant.

Thank you again for the opportunity to review this code.

@harjotgill harjotgill added this pull request to the merge queue Mar 14, 2023
Merged via the queue into main with commit b8da2c3 Mar 14, 2023
@harjotgill harjotgill deleted the max_files branch March 14, 2023 01:41
shoyuf pushed a commit to maybeLab/openai-mr-reviewer-gltlab that referenced this pull request Jun 25, 2023
Bumps [chatgpt](https://github.com/transitive-bullshit/chatgpt-api) from 4.8.3 to 5.0.4.
- [Release notes](https://github.com/transitive-bullshit/chatgpt-api/releases)
- [Commits](transitive-bullshit/agentic@v4.8.3...v5.0.4)

---
updated-dependencies:
- dependency-name: chatgpt
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
shoyuf pushed a commit to maybeLab/openai-mr-reviewer-gltlab that referenced this pull request Jun 25, 2023
…itai#27)

<!-- This is an auto-generated comment: release notes by openai -->
### Summary by OpenAI

Purpose: Limit the number of files that can be reviewed by this tool.

- New Feature: Added a new input option `max_files` to specify the
maximum number of files that can be reviewed.
- Bug fix: Added checks to ensure that the number of files being
reviewed does not exceed this limit.
- Refactor: Updated the code to improve readability and maintainability.
<!-- end of auto-generated comment: release notes by openai -->
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant