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

santize suggestions #199

Merged
merged 1 commit into from
Apr 19, 2023
Merged

santize suggestions #199

merged 1 commit into from
Apr 19, 2023

Conversation

harjotgill
Copy link
Contributor

@harjotgill harjotgill commented Apr 19, 2023

Summary by OpenAI

Release Notes

  • New Feature: Added sanitizeComment function to remove line numbers from suggestion blocks in comments and uses it to sanitize comments before storing them as reviews.

"Clean code is a delight, with sanitized comments shining bright 🌟. Bugs and errors take flight, with this PR's changes in sight 🐛✈️."

@github-actions
Copy link
Contributor

🤖 OpenAI

Review of Pull Request #X

The changes in this pull request aim to improve the quality and maintainability of the codebase. The main change is the addition of a sanitizeComment function that removes line numbers from suggestion blocks in comments. This function is used to sanitize comments before storing them as reviews.

This change improves the consistency of the review process by ensuring that all comments are stored in a standardized format. It also enhances the security of the application by removing any potentially sensitive information, such as line numbers, from the comments.

File Summary
src/review.ts Adds a sanitizeComment function to remove line numbers from suggestion blocks in comments and uses it to sanitize comments before storing them as reviews.

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

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.

Files ignored due to filter (1)

Ignored files

  • dist/index.js

@harjotgill harjotgill merged commit b73fb59 into main Apr 19, 2023
@harjotgill harjotgill deleted the sanitize_lno branch April 19, 2023 00:17
shoyuf pushed a commit to maybeLab/openai-mr-reviewer-gltlab that referenced this pull request Jun 25, 2023
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