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

incrementally summarize #211

Merged
merged 7 commits into from
Apr 20, 2023
Merged

incrementally summarize #211

merged 7 commits into from
Apr 20, 2023

Conversation

harjotgill
Copy link
Contributor

@harjotgill harjotgill commented Apr 20, 2023

Summary by OpenAI

Release Notes:

  • New Feature: The bot can now summarize changes in pull requests incrementally, retrieve the highest reviewed commit ID, and review only the changed files since then. It also generates release notes and comments on the pull request.
  • Refactor: Several improvements were made to the codebase, including modifications to the action inputs, enhancements to the Commenter class, and updates to the Options class.

"Code improved, features new,
Reviewing PRs like a pro, it'll do.
Incremental summaries, release notes too,
OpenAI's bot, always coming through."

@github-actions
Copy link
Contributor

github-actions bot commented Apr 20, 2023

🤖 OpenAI

High-level Summary

This pull request introduces several improvements to the OpenAI ChatGPT-based PR reviewer and summarizer. The changes include updates to the README file, modifications to the action inputs, and enhancements to the codebase. Notably, the bot now has the ability to summarize changes in pull requests incrementally, retrieve the highest reviewed commit ID, and review only the changed files since then. It also generates release notes and comments on the pull request.

Files Summary

File Summary
README.md Minor updates to the text, such as adding hyphens and clarifying notes. Two new bullet points were added to the Overview section.
action.yml Adds an input to limit the number of files to summarize and review, renames a few inputs, and updates their descriptions. Modifies the default prompts for summarizing and final summarization response.
src/commenter.ts Adds methods to extract and remove content within HTML tags, and use them to modify the get_description and get_release_notes methods in Commenter. A new method get_raw_summary is also added.
src/main.ts Changes the input parameter names for max_files_to_summarize and max_files_to_review to a single parameter called max_files.
src/options.ts Renames the summary field in Inputs class to raw_summary. Merges the max_files_to_summarize and max_files_to_review fields in Options class into a single max_files field.
src/review-comment.ts Removes unused code, packs file content and diff into inputs, and adds a summary of the PR to inputs if it's not too long.
src/review.ts Adds a feature to summarize changes in pull requests incrementally. Retrieves the highest reviewed commit ID and reviews only the changed files since then. Summarizes the summaries of changed files, deduplicates them, and groups related changes together. Generates release notes and comments on the pull request.

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

src/main.ts Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
src/main.ts Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
action.yml Show resolved Hide resolved
action.yml Show resolved Hide resolved
action.yml Show resolved Hide resolved
action.yml Show resolved Hide resolved
README.md Show resolved Hide resolved
action.yml Show resolved Hide resolved
@harjotgill harjotgill enabled auto-merge (squash) April 20, 2023 03:55
@harjotgill harjotgill merged commit f63fbce into main Apr 20, 2023
@harjotgill harjotgill deleted the summary branch April 20, 2023 03:55
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