Skip to content

Implementing copy/paste - #57262

Merged
Sheetal Nandi (sheetalkamat) merged 53 commits into
microsoft:mainfrom
navya9singh:testingCopyPaste
May 8, 2024
Merged

Implementing copy/paste#57262
Sheetal Nandi (sheetalkamat) merged 53 commits into
microsoft:mainfrom
navya9singh:testingCopyPaste

Conversation

@navya9singh

@navya9singh navya9singh commented Feb 1, 2024 •
edited
Loading

Copy link
Copy Markdown
Member

Fixes #50187

@typescript-bot TypeScript Bot (typescript-bot) added Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug labels Feb 1, 2024
@typescript-bot

TypeScript Bot (typescript-bot) commented Feb 1, 2024

Copy link
Copy Markdown
Contributor

Thanks for the PR! It looks like you've changed the TSServer protocol in some way. Please ensure that any changes here don't break consumers of the current TSServer API. For some extra review, we'll ping Sheetal Nandi (@sheetalkamat), Matt Bierner (@mjbvz), Kat Marchán (@zkat), and Joaquin Jares (@joj) for you. Feel free to loop in other consumers/maintainers if necessary.

navya9singh
navya9singh commented Feb 1, 2024
View reviewed changes
Comment thread src/services/codefixes/importFixes.ts
navya9singh
navya9singh commented Feb 1, 2024
View reviewed changes
Comment thread src/harness/client.ts Outdated
navya9singh
navya9singh commented Feb 1, 2024
View reviewed changes
Comment thread tests/cases/fourslash/server/postPasteImportFix_multiplePastes2.ts Outdated
mjbvz
Matt Bierner (mjbvz) reviewed Feb 5, 2024
View reviewed changes

@mjbvz Matt Bierner (mjbvz) left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just reviewed the protocol changes. Direction looks good but had a few suggestions for using different types

Comment thread src/server/protocol.ts Outdated
Comment thread src/server/protocol.ts Outdated
Comment thread src/server/protocol.ts Outdated
sheetalkamat
Sheetal Nandi (sheetalkamat) reviewed Feb 6, 2024
View reviewed changes
Comment thread src/server/project.ts Outdated
zkat
Kat Marchán (zkat) reviewed Feb 6, 2024
View reviewed changes
Comment thread src/services/postPasteImportFix.ts Outdated
Matt Bierner (mjbvz) pushed a commit to microsoft/vscode that referenced this pull request Feb 7, 2024
Matt Bierner
Hook up prototype paste with imports for JS/TS
c85c0ed 
For microsoft/TypeScript#57262 but with proposed changes to ts protocol
@mjbvz Matt Bierner (mjbvz) mentioned this pull request Feb 7, 2024
Merged
@typescript-bot

TypeScript Bot (typescript-bot) commented Feb 12, 2024

Copy link
Copy Markdown
Contributor

Looks like you're introducing a change to the public API surface area. If this includes breaking changes, please document them on our wiki's API Breaking Changes page.

Also, please make sure Daniel Rosenwasser (@DanielRosenwasser) and Ryan Cavanaugh (@RyanCavanaugh) are aware of the changes, just as a heads up.

@navya9singh
navya9singh marked this pull request as ready for review February 12, 2024 23:54
andrewbranch
Andrew Branch (andrewbranch) reviewed May 1, 2024
View reviewed changes
Comment thread src/services/codefixes/importFixes.ts Outdated
andrewbranch
Andrew Branch (andrewbranch) reviewed May 2, 2024
View reviewed changes
Comment thread tests/cases/fourslash/server/pasteEdits_knownSourceFile.ts Outdated
Comment thread src/server/protocol.ts Outdated
Comment thread tests/cases/fourslash/server/pasteEdits_existingImports1.ts Outdated
sheetalkamat
Sheetal Nandi (sheetalkamat) reviewed May 2, 2024
View reviewed changes
Comment thread src/testRunner/unittests/tsserver/pasteEdits.ts Outdated
sheetalkamat
Sheetal Nandi (sheetalkamat) reviewed May 2, 2024
View reviewed changes
Comment thread src/testRunner/unittests/tsserver/pasteEdits.ts Outdated
sheetalkamat
Sheetal Nandi (sheetalkamat) reviewed May 2, 2024
View reviewed changes
Comment thread tests/cases/fourslash/server/pasteEdits_existingImports1.ts Outdated
@sheetalkamat
Sheetal Nandi (sheetalkamat) dismissed Daniel Rosenwasser (DanielRosenwasser)’s stale review May 8, 2024 23:00

Trying to dismiss this to see if it enables merging. Right now its blocked

@sheetalkamat

Sheetal Nandi (sheetalkamat) commented May 8, 2024

Copy link
Copy Markdown
Member

Daniel Rosenwasser (@DanielRosenwasser) navya9singh wasnt able to merge because it was marked as pending your review and you are oof. And the feedback seems to be addressed

@sheetalkamat
Sheetal Nandi (sheetalkamat) merged commit be8fb98 into microsoft:main May 8, 2024
Closed
Closed
@microsoft Microsoft (microsoft) locked as resolved and limited conversation to collaborators Oct 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@andrewbranch Andrew Branch (andrewbranch) andrewbranch approved these changes
@sheetalkamat Sheetal Nandi (sheetalkamat) sheetalkamat approved these changes
@DanielRosenwasser Daniel Rosenwasser (DanielRosenwasser) Awaiting requested review from DanielRosenwasser
+2 more reviewers
@zkat Kat Marchán (zkat) zkat left review comments
@mjbvz Matt Bierner (mjbvz) mjbvz approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@navya9singh navya9singh

Labels

Author: Team For Milestone Bug PRs that fix a bug with a specific milestone

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Bring along imports when copy and pasting code across files

8 participants

@navya9singh @typescript-bot @sheetalkamat @zkat @DanielRosenwasser @andrewbranch @mjbvz @sandersn