Skip to content

fix: write docstring always #6159

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

lyt213
Copy link

@lyt213 lyt213 commented Jun 17, 2025

Description

issue-2850306930
current: In VSCode, there is a function called "write a doc string" in the right-click menu, which passes a parameter "onlyOneInsertion" to streamEdit, which only allows receiving up to one line of duplicate content.
image

Problem: Except for the first line, all other content will be delete by function reapplyWithMyersDiff,cause by wrong range
image

fixed:
when params onlyOneInsertion is true,set endrange = startrange

[ What changed? Feel free to be brief. ]

Checklist

  • I've read the contributing guide
  • The relevant docs, if any, have been updated or created
  • The relevant tests, if any, have been updated or created

Screenshots

[ For visual changes, include screenshots. Screen recordings are particularly helpful, and appreciated! ]

Tests

[ What tests were added or updated to ensure the changes work as expected? ]

@lyt213 lyt213 requested a review from a team as a code owner June 17, 2025 03:02
@lyt213 lyt213 requested review from RomneyDa and removed request for a team June 17, 2025 03:02
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jun 17, 2025
Copy link

netlify bot commented Jun 17, 2025

Deploy Preview for continuedev canceled.

Name Link
🔨 Latest commit 32b87f0
🔍 Latest deploy log https://app.netlify.com/projects/continuedev/deploys/6850de81d827c200087ab7ff

Copy link

github-actions bot commented Jun 17, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

Copy link

recurseml bot commented Jun 17, 2025

✨ No issues found! Your code is sparkling clean! ✨

@lyt213
Copy link
Author

lyt213 commented Jun 17, 2025

I have read the CLA Document and I hereby sign the CLA

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

1 participant