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

post review only when comments are there #379

Merged
merged 1 commit into from
Jul 24, 2023
Merged

post review only when comments are there #379

merged 1 commit into from
Jul 24, 2023

Conversation

harjotgill
Copy link
Contributor

@harjotgill harjotgill commented Jul 24, 2023

Summary by OpenAI

Chore:

  • Added an early return in submitReview function of src/commenter.ts when there are no review comments, improving efficiency by avoiding unnecessary operations.

🎉 Here's to the code that's lean and mean,
No more wasted cycles, it's a well-oiled machine. 🚀
When comments are none, it knows what to shun,
Celebrating small wins, this battle is won! 🏆

@github-actions
Copy link
Contributor

🤖 OpenAI

The changeset introduces an optimization in the submitReview function within src/commenter.ts. It adds a condition to return early if there are no review comments, preventing unnecessary operations.

File Summary
src/commenter.ts The change adds an early return in the submitReview function when there are no review comments. This prevents unnecessary operations when the comment buffer is empty.

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.


In the recent run, only the files that changed from the base of the PR and between f1cf28d89d8624f97766595089532a7902a57ac8 and 89e2ef09af60bab18eed9df68abe8ba9c397739f commits were reviewed.

@harjotgill harjotgill merged commit 955c447 into main Jul 24, 2023
@harjotgill harjotgill deleted the comments branch July 24, 2023 18:45
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