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

Update prompts #408

Merged
merged 11 commits into from
Aug 1, 2023
Merged

Update prompts #408

merged 11 commits into from
Aug 1, 2023

Conversation

harjotgill
Copy link
Contributor

@harjotgill harjotgill commented Aug 1, 2023

Summary by CodeRabbit

New Feature:

  • Introduced a Mobile struct in mobile.go to simulate the behavior of a mobile device with methods for turning on/off, usage, and charging.
  • Enhanced code snippets and instructions for providing replacement code suggestions in src/prompts.ts.
  • Improved parsing functionality in src/review.ts to handle context lines and skip annotations for the first 3 and last 3 lines.
  • Added a new section in the PR description promoting the Pro version of the project.

🎉 Code's evolving, growing so fine, 🌱

With every pull request, it shines. ✨

Mobile simulation, parsing delight, 📱🎁

Our codebase takes an exciting flight! 🚀

@github-actions
Copy link
Contributor

github-actions bot commented Aug 1, 2023

Image description CodeRabbit

Walkthrough: This update introduces a new Mobile struct in the Go codebase, simulating basic operations of a mobile device. In the TypeScript project, it refines the line number range format and enhances the patch parsing function to handle context lines more effectively. It also includes promotional content for the Pro version of the project.

Changes:

Files Summary
mobile.go Introduced a Mobile struct representing a mobile device with methods for turning on/off, using, and charging the device. The main function demonstrates these operations.
src/prompts.ts, src/review.ts Updated line number range format in prompts.ts. Enhanced parsePatch function in review.ts to handle context lines better. Added promotional content for the Pro version of the project.

Notes

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.

Ignoring further reviews

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

Support us :)

If you like this project, please support us by purchasing the Pro version! The Pro version has advanced context and several proprietary improvements compared to the open source version.

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 6e65620 and d5f436d commits.
Files selected (3)
  • mobile.go (1)
  • src/prompts.ts (2)
  • src/review.ts (3)
Files ignored due to filter (1)
  • dist/index.js
Review comments generated (6)
  • Review: 4
  • LGTM: 2

src/prompts.ts Outdated Show resolved Hide resolved
src/prompts.ts Show resolved Hide resolved
mobile.go Outdated Show resolved Hide resolved
src/review.ts 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 d5f436d and 315476a commits.
Files selected (2)
  • mobile.go (1)
  • src/prompts.ts (2)
Files ignored due to filter (1)
  • dist/index.js
Files skipped from review due to trivial changes (1)
  • src/prompts.ts
Review comments generated (7)
  • Review: 3
  • LGTM: 4

mobile.go Outdated Show resolved Hide resolved
mobile.go Outdated Show resolved Hide resolved
mobile.go Outdated Show resolved Hide resolved
@harjotgill harjotgill merged commit 9a257da into main Aug 1, 2023
@harjotgill harjotgill deleted the suggestions_3 branch August 1, 2023 20:09
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