Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Command arguments are messed up #27535

Closed
jrieken opened this issue May 30, 2017 · 0 comments
Closed

Command arguments are messed up #27535

jrieken opened this issue May 30, 2017 · 0 comments
Assignees
@sandy081
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Milestone
May 2017

Comments

@jrieken
Copy link
Member

jrieken commented May 30, 2017

re #27467

The API allows me to have a real Command with arguments but they get lost in massageTreeItem. You should use ExtHostCommands#converter to get full support for this
@jrieken jrieken added the bug Issue identified by VS Code Team member as probable bug label May 30, 2017
@sandy081 sandy081 mentioned this issue May 30, 2017
Closed
7 tasks
@sandy081 sandy081 added this to the May 2017 milestone May 30, 2017
@jrieken jrieken added the verified Verification succeeded label Jun 1, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@sandy081 sandy081

Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Projects
None yet
Milestone
May 2017
Development

No branches or pull requests
2 participants
@jrieken @sandy081