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

don't send entire file when reviewing #263

Merged
merged 9 commits into from
Apr 28, 2023
Merged

don't send entire file when reviewing #263

merged 9 commits into from
Apr 28, 2023

Conversation

harjotgill
Copy link
Contributor

@harjotgill harjotgill commented Apr 28, 2023

Summary by OpenAI

Release Notes

  • Improvement: Updated constants, increased limits, improved message formatting, added error handling, and logging statements to enhance the performance, security, and maintainability of the system.

"Codebase refined,
Performance enhanced,
Bugs squashed,
Celebrate with a 🎉!"

@github-actions
Copy link
Contributor

github-actions bot commented Apr 28, 2023

🤖 OpenAI

Summary of Changes

The pull request includes several changes to improve the functionality and usability of the codebase. The changes include updating constants, increasing limits, improving message formatting, adding error handling, and logging statements. Overall, the changes seem to be focused on enhancing the performance, security, and maintainability of the system.

File Summary
commenter.ts Updates the DESCRIPTION_START_TAG constant to use a template literal and modifies the newDescription variable to append the release notes message without adding an extra newline.
limits.ts Increases the maxTokens limit from 3900 to 4000 and provides a margin of 100 tokens for requestTokens.
prompts.ts Improves the format and content of messages that are sent during a GitHub pull request review. Suggestions include using constants instead of hardcoding strings, adding error handling, and using a template engine to generate message content.
review.ts Adds logging statements to the code and includes minor changes to the codeReview and summarize functions. It also adds a check to limit the number of patches sent in a single request.

Overall, the changes seem to be well-considered and focused on improving the quality and maintainability of the codebase. The suggestions provided for each file should help to further enhance the functionality and usability of the system.


Chat with 🤖 OpenAI Bot (@openai)

  • Reply on review comments left by this bot to ask follow-up questions. A review comment is a comment on a diff or a file.
  • Invite the bot into a review comment chain by tagging @openai in a reply.

Code suggestions

  • The bot may make code suggestions, but please review them carefully before committing since the line number ranges may be misaligned.
  • You can edit the comment made by the bot and manually tweak the suggestion if it is slightly off.

Ignoring further reviews

  • Type @openai: ignore anywhere in the PR description to ignore further reviews from the bot.

Files ignored due to filter (1)

Ignored files

  • dist/index.js

In the recent run, only the files that changed from the base of the PR and between 60b7c638c913b8f7e8631e9773811a34da5d00c9 and 219b19745f239124a0d8b043b2801146b11f7be6 commits were reviewed.

Files not reviewed due to simple changes (2)

Skipped review in the recent run

  • src/commenter.ts
  • src/review.ts

@harjotgill harjotgill merged commit bad0f55 into main Apr 28, 2023
@harjotgill harjotgill deleted the skip_file branch April 28, 2023 20:05
shoyuf pushed a commit to maybeLab/openai-mr-reviewer-gltlab that referenced this pull request Jun 25, 2023
<!-- This is an auto-generated comment: release notes by openai -->
### Summary by OpenAI

## Release Notes

- **Improvement:** Updated constants, increased limits, improved message
formatting, added error handling, and logging statements to enhance the
performance, security, and maintainability of the system.

> "Codebase refined,  
> Performance enhanced,  
> Bugs squashed,  
> Celebrate with a 🎉!"
<!-- 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