Repository navigation
feat(lsp): complete closing triple-quoted strings - #4737
mangeshraut712 wants to merge 13 commits into
Conversation
|
Hi @mangeshraut712! Thank you for your pull request and welcome to our community. Action RequiredIn order to merge any pull request (code, docs, etc.), we require contributors to sign our Contributor License Agreement, and we don't seem to have one on file for you. ProcessIn order for us to review and merge your suggested changes, please sign at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need to sign the corporate CLA. Once the CLA is signed, our tooling will perform checks and validations. Afterwards, the pull request will be tagged with If you have received this in error or have any questions, please contact us at cla@meta.com. Thanks! |
|
Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Meta Open Source project. Thanks! |
e7b7748 to
55ebe4a
Compare
|
This pull request has been imported. If you are a Meta employee, you can view this in D119010817. (Because this pull request was imported automatically, there will not be any future comments.) |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
3bb0928 to
dd4e429
Compare
dd4e429 to
cff85b2
Compare
This comment has been minimized.
This comment has been minimized.
|
CLA is signed and the diff has been imported. mypy_primer reports no type-check changes on the corpus. This completes closing triple-quoted strings in the LSP. Could a maintainer review when you have a moment? |
febdd22 to
e43705b
Compare
This comment has been minimized.
This comment has been minimized.
BSD head rejects -n 0; use dd bs=1 count=0 so the scrut pipe-close examples stay portable (same as meta#5390).
This comment has been minimized.
This comment has been minimized.
# Conflicts: # pyrefly/lib/test/lsp/completion.rs
This comment has been minimized.
This comment has been minimized.
…string-completions
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
According to mypy_primer, this change doesn't affect type check results on a corpus of open source code. ✅ |
Summary
Fixes #4346.
When the cursor is immediately after exactly three matching quotes (
"""or'''), offer atriple-quoted stringcompletion that inserts the matching closer after the cursor. When the client supports snippets, the cursor placeholder is placed before the closer.r,u,f,b,fr,rf,br, andrb.AI usage: an assistant drafted the completion helper and tests; I reviewed the LSP edit ranges, prefix handling, and test cursors.
Test Plan