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 the prompt output format #133

Merged
merged 1 commit into from
May 23, 2024

Conversation

sauravpanda
Copy link
Member

@sauravpanda sauravpanda commented May 23, 2024

PR Description

This pull request introduces changes to the prompt output format and refactors the code to include new features for generating feedback.

Significant Modifications

  • Updated the prompt output format in output.py to include start and end line numbers and file name.
  • Refactored the code in prompts.py to include start and end line numbers and file name for each comment.
  • Added instructions for generating all possible feedbacks and providing actionable points for each comment.
  • Improved the clarity and specificity of feedback by referencing specific file(s) and line number(s) of code being addressed for each comment.
  • Utilized markdown code blocks to quote relevant snippets of code when necessary for better readability.

✨ Generated with love by Kaizen ❤️

Original Description None

@sauravpanda sauravpanda linked an issue May 23, 2024 that may be closed by this pull request
Copy link
Contributor

kaizen-bot bot commented May 23, 2024

Code Review Feedback

Code Quality

[moderate] -> The 'PR_COLLAPSIBLE_TEMPLATE' variable is defined but not used anywhere in the code.
[moderate] -> The 'DESC_COLLAPSIBLE_TEMPLATE' variable is defined but not used anywhere in the code.

Potential Issues

[low] -> There are inconsistent extra spaces at the end of lines in the 'PR_COLLAPSIBLE_TEMPLATE' and 'DESC_COLLAPSIBLE_TEMPLATE' definitions.

Improvements

[low] -> The naming convention for the 'start_line' and 'end_line' variables in the JSON object should be consistent with snake_case.

✨ Generated with love by Kaizen ❤️

@sauravpanda sauravpanda merged commit cadfd1e into main May 23, 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.

Add spacing in code review response formation
1 participant