feat: add copy selection as context feature#11
Conversation
yinyu985
force-pushed
the
feat/copy-selection-context
branch
from
5bf664e to
de6589b
Compare
| "watch": "npm run esbuild-base -- --watch" | ||
| }, | ||
| "version": "0.8.0" | ||
| "version": "0.8.3" |
There was a problem hiding this comment.
No need to set the version; this is set by release CI.
| "title": "Copy Folder Structure" | ||
| }, | ||
| { | ||
| "category": "Copy Context", |
| { | ||
| "category": "Copy Context", | ||
| "command": "extension.copySelection", | ||
| "title": "Copy Selection as Context" |
There was a problem hiding this comment.
We can simplify the title here, let's just use Copy Selection
yinyu985
force-pushed
the
feat/copy-selection-context
branch
from
de6589b to
4189051
Compare
|
Hi @Fralleee,I've successfully resolved the merge conflicts by
rebasing my branch on the latest upstream/main. The conflicts occurred
because I modified existing files (package.json, extension.ts) in
addition to adding new files, and these files had also been updated in
the upstream since I created my branch.All functionality is now
working correctly:- The new "Copy Selection as Context" feature is
fully implemented- All existing functionality remains intact- Import
paths have been updated to match the recent rename from "copy-context"
to "copy-content"The PR is now ready for review and merge. Thanks for
considering this feature!
Roland Chelwing-Grzybowski ***@***.***> 于2025年10月28日周二
22:26写道：
… *Fralleee* left a comment (Fralleee/copy-context#11)
<#11 (comment)>
Hey @yinyu985 <https://github.com/yinyu985>.
Good feature proposal!
Can you fix the conflicts and I'll happily merge this 👍
—
Reply to this email directly, view it on GitHub
<#11 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AMMVURLEDMJISCLWKJEY3S33Z54KPAVCNFSM6AAAAACKJ5TJTCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTINJWG43TGMJYGQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
|
Hi @Fralleee Thanks for the detailed feedback! I've addressed all your suggestions: ✅ Version: Reverted to 0.8.0 as you noted that version is managed by release CI ✅ Menu configuration: Fixed the menu placement - tab copy commands remain in ✅ Command title: Removed the category and simplified the title to just "Copy Selection" All functionality is working correctly and the PR should now be ready for merge. Thanks again for your guidance on this! 👍 |
|
🎉 This PR is included in version 0.10.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
2 participants
Closes #10