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

separate config for max files to summarize and review #83

Merged
merged 5 commits into from
Mar 24, 2023
Merged

Conversation

harjotgill
Copy link
Contributor

@harjotgill harjotgill commented Mar 24, 2023

Summary by OpenAI

Chore: Add separate configuration options for maximum files to summarize and review. This pull request updates the Options object and class with two new properties, max_files_to_summarize and max_files_to_review, and renames the existing max_files property to max_files_to_summarize. It also updates tsconfig.json by removing some options and adding others, including setting the target and module to "ESNext" and enabling strict mode.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 24, 2023

🤖 OpenAI

Thank you for letting me participate in the code review. Based on my analysis, this pull request adds separate configuration options for maximum files to summarize and review, with default values updated accordingly. It also updates the Options object and class with two new properties, max_files_to_summarize and max_files_to_review, and renames the existing max_files property to max_files_to_summarize. Additionally, it updates tsconfig.json by removing some options and adding others, including setting the target and module to "ESNext" and enabling strict mode.

Here is a table summarizing the changes made in each file:

File Summary
action.yml Updates the inputs section of action.yml to add separate configuration options for maximum files to summarize and review. Default values are also updated.
src/main.ts
src/options.ts
Adds two new properties to the Options object and class, max_files_to_summarize and max_files_to_review, and renames the existing max_files property to max_files_to_summarize. It also updates the constructor and the initialization of these properties.
tsconfig.json Updates tsconfig.json by removing some options and adding others, including setting the target and module to "ESNext" and enabling strict mode.

Skipped files due to filter (1)

  • dist/index.js

Chatting 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.

src/review.ts Outdated Show resolved Hide resolved
@github-actions
Copy link
Contributor

🤖 OpenAI

Reviewing existing code in: tsconfig.json

The existing code in tsconfig.json looks correct to me. No issues found.

@harjotgill harjotgill merged commit 009542a into main Mar 24, 2023
@harjotgill harjotgill deleted the max_files branch March 24, 2023 20:21
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

Chore: Add separate configuration options for maximum files to summarize
and review. This pull request updates the `Options` object and class
with two new properties, `max_files_to_summarize` and
`max_files_to_review`, and renames the existing `max_files` property to
`max_files_to_summarize`. It also updates `tsconfig.json` by removing
some options and adding others, including setting the target and module
to "ESNext" and enabling strict mode.
<!-- 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