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

prompt updates #412

Merged
merged 2 commits into from
Aug 3, 2023
Merged

prompt updates #412

merged 2 commits into from
Aug 3, 2023

Conversation

harjotgill
Copy link
Contributor

@harjotgill harjotgill commented Aug 3, 2023

Summary by CodeRabbit

### Refactor:
- Removed a section of code related to notes in `src/review.ts`.
- Added a new section with tips in `src/review.ts`.
- Modified the `sanitizeComment` function to handle different types of code blocks.

---

> 🎉 Code's been refactored, it's no jest,
> 
> With tips and tricks, it's now the best.
> 
> Sanitize comments, keep them clean,
> 
> In our codebase, no mess to be seen! 🧹🌟

@github-actions
Copy link
Contributor

github-actions bot commented Aug 3, 2023

Image description CodeRabbit

Walkthrough

The changeset primarily focuses on enhancing the user experience by replacing a section related to notes with a more informative tips section. Additionally, it improves the sanitizeComment function to handle various types of code blocks, ensuring better data sanitization and security.

Changes

File Summary
src/review.ts The diff removes a section of code related to notes and introduces a new section with tips. It also refines the sanitizeComment function to manage different types of code blocks without altering the existing logic or functionality.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Image description CodeRabbit

Commits Files that changed from the base of the PR and between a1729bc and 8db3d18 commits.
Files selected (2)
  • src/prompts.ts (1)
  • src/review.ts (3)
Files ignored due to filter (1)
  • dist/index.js
Files not summarized due to errors (1)
  • src/prompts.ts (diff tokens exceeds limit)
Files skipped from review due to trivial changes (1)
  • src/review.ts
Review comments generated (4)
  • Review: 3
  • LGTM: 1

Tips

Chat with Image description CodeRabbit Bot (@coderabbitai)

  • 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 @coderabbitai 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.

Pausing incremental reviews

  • Add @coderabbitai: ignore anywhere in the PR description to pause further reviews from the bot.

src/prompts.ts Show resolved Hide resolved
src/prompts.ts Outdated Show resolved Hide resolved
src/prompts.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Image description CodeRabbit

Commits Files that changed from the base of the PR and between 8db3d18 and 118da9d commits.
Files selected (1)
  • src/prompts.ts (1)
Files ignored due to filter (1)
  • dist/index.js
Files not summarized due to errors (1)
  • src/prompts.ts (diff tokens exceeds limit)
Review comments generated (4)
  • Review: 2
  • LGTM: 2

Tips

Chat with Image description CodeRabbit Bot (@coderabbitai)

  • 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 @coderabbitai 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.

Pausing incremental reviews

  • Add @coderabbitai: ignore anywhere in the PR description to pause further reviews from the bot.

src/prompts.ts Show resolved Hide resolved
src/prompts.ts Show resolved Hide resolved
@harjotgill harjotgill merged commit 74543a3 into main Aug 3, 2023
@harjotgill harjotgill deleted the port branch August 3, 2023 01:11
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