Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat updated pr review output format #446

Conversation

sauravpanda
Copy link
Member

@sauravpanda sauravpanda commented Aug 17, 2024

Add Model Selection and Minor Code Review Improvements

Overview

This pull request introduces the ability to select different models for code review and refines the code review format for better clarity.

Changes

  • Key Changes:
    • Added model selection to the process_pr function, allowing users to specify the desired model for code review.
    • Updated the config.json file to include model information for the "best" model.
    • Modified the code_review_formatter.py to improve the presentation of review results, separating critical issues from refinement suggestions and providing a more concise format for minor notes.
  • New Features:
    • None
  • Refactoring:
    • Refactored the code_review_formatter.py to improve the organization and readability of the code review output.

✨ Generated with love by Kaizen ❤️

Original Description

Copy link
Contributor

kaizen-bot bot commented Aug 17, 2024

Code Review

Attention Required: This PR has potential issues. 🚨

Configuration

Changes made to sensitive file

Potential Solution:

NA

config.json | 11 - 11

reason_for_request: Changes were made to config.json, which needs review

level: [critical] , severity: [10]

✨ Generated with love by Kaizen ❤️


Useful Commands
  • Feedback: Reply with !feedback [your message]

  • Ask PR: Reply with !ask-pr [your question]

  • Review: Reply with !review

@@ -11,7 +11,17 @@
"input_cost_per_token": 0.000000015,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment: Changes made to sensitive file

Solution: NA

Reason For Comment: Changes were made to config.json, which needs review

Copy link
Contributor

kaizen-bot bot commented Aug 17, 2024

Code Review

Attention Required: This PR has potential issues. 🚨

Configuration Management

The use of 'os.environ' for API keys is correct, but ensure that environment variables are loaded correctly.

Potential Solution:

Add a check to ensure that the required environment variables are present.

config.json | 13 - 14

reason_for_request: If environment variables are not set, it can lead to application failures.

level: [critical] , severity: [10]

Configuration

Changes made to sensitive file

Potential Solution:

NA

config.json | 11 - 11

reason_for_request: Changes were made to config.json, which needs review

level: [critical] , severity: [10]

✨ Generated with love by Kaizen ❤️


Useful Commands
  • Feedback: Reply with !feedback [your message]

  • Ask PR: Reply with !ask-pr [your question]

  • Review: Reply with !review

config.json Show resolved Hide resolved
config.json Show resolved Hide resolved
@sauravpanda sauravpanda merged commit bae496e into main Aug 17, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant