Supply multiselects to scm/resourceGroup/context menu commands (fix #92337)
#192172
Conversation
|
/assign @lszomoru |
|
@lszomoru please consider adding this to the October 2023 milestone. |
|
Can this be merged soon? Or at least milestoned? |
|
@lszomoru I have done the merge necessary to bring this branch up to date with man, and have re-tested my changes. If possible please review and merge in time for next week's endgame. |
|
Updated from main and ready again for review & merging 🤞 |
|
@lszomoru I have updated from main and resolved the conflict. Please review. |
|
@lszomoru merging this would bring a big benefit for the SCM extension I maintain. Please consider it soon. |
|
@lszomoru any chance for April? |
|
Please accept this PR to close the linked issue, which is tagged |
|
@lszomoru please consider merging this. I have an extension that would benefit from it. |
|
Please, review this minimal change. |
Maybe for August? 🙏 |
|
@lszomoru I have merged the ESM changes from main and re-tested. Please review 🙏 |
|
@lszomoru please can we have this in 1.95? |
|
@lszomoru Before I spend the time to resolve conflicts and update this branch I'd like to know what the chances are of you reviewing the PR. |
|
@lszomoru I have resolved the conflict and updated this branch from main, so it's now ready again for your consideration. |
This PR fixes #92337
One way to verify is to clone https://github.com/gjsjohnmurray/vscode-extension-samples/tree/show-vscode-92337 which modifies source-control-sample to add a second resource group, and contributes a command to
scm/resourceGroup/context(both inline and on a context menu) that shows a notification of the label(s) of the selected group(s).