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

remove sanitization code for now #198

Merged
merged 1 commit into from
Apr 18, 2023
Merged

remove sanitization code for now #198

merged 1 commit into from
Apr 18, 2023

Conversation

harjotgill
Copy link
Contributor

@harjotgill harjotgill commented Apr 18, 2023

Summary by OpenAI

Release Notes

Bug fix: The code removes a function that sanitizes line numbers from suggestion comments in the parseReview function. The storeReview function no longer calls this removed function and instead stores the comment as is.

"Code changes made with care,
Bugs fixed, we're now aware.
With each pull request we grow,
Our codebase stronger than before."

@harjotgill harjotgill enabled auto-merge (squash) April 18, 2023 23:38
@github-actions
Copy link
Contributor

🤖 OpenAI

Summary of Changes

The changes made to the codebase involve removing a function that sanitizes line numbers from suggestion comments in the parseReview function. The storeReview function no longer calls this removed function and instead stores the comment as is.

This change may have been made to simplify the codebase by removing unnecessary sanitization logic. However, it is important to ensure that this does not introduce any security vulnerabilities or other issues related to untrusted user input.

File Summary
src/review.ts The code removes a function that sanitizes line numbers from suggestion comments in the parseReview function. The storeReview function no longer calls this removed function and instead stores the comment as is.

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 a5a1bf3 into main Apr 18, 2023
@harjotgill harjotgill deleted the sanitize_remove branch April 18, 2023 23:38
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