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

refactor: single suggestion per review section #174

Merged
merged 2 commits into from
Apr 12, 2023
Merged

Conversation

harjotgill
Copy link
Contributor

@harjotgill harjotgill commented Apr 12, 2023

Summary by OpenAI

Release Notes

  • Refactor: The review process has been updated to allow only one replacement suggestion or new code snippet per review section, and the instructions for providing replacement suggestions have been clarified.

"Code reviews made better,
One suggestion at a time.
Clarity and consistency,
Our code now sublime."

@github-actions
Copy link
Contributor

🤖 OpenAI

Review of Pull Request #1234

This pull request refactors the review process to improve the clarity and consistency of code reviews. Specifically, it limits the number of replacement suggestions or new code snippets per review section to one, and provides clearer instructions for providing replacement suggestions.

File Summary
src/review.ts The review process has been updated to allow only one replacement suggestion or new code snippet per review section, and the instructions for providing replacement suggestions have been clarified.

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.

Files ignored due to filter (1)

Ignored files

  • dist/index.js

@harjotgill harjotgill merged commit 59fec0a into main Apr 12, 2023
@harjotgill harjotgill deleted the single_snippet branch April 12, 2023 18:47
shoyuf pushed a commit to maybeLab/openai-mr-reviewer-gltlab that referenced this pull request Jun 25, 2023
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