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.

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

Sort @workspace /explain above @terminal /explain #213482

Merged
merged 1 commit into from
May 26, 2024

Conversation

roblourens
Copy link
Member

Fix microsoft/vscode-copilot-release#1245

This is a bit of a hack but has some advantanges over adding an 'order' field to the chat participant or something like that. We can keep the order the same in other cases except for when the user is typing to filter the / list, then @Terminal slash commands will be pushed down.

Fix microsoft/vscode-copilot-release#1245

This is a bit of a hack but has some advantanges over adding an 'order' field to the chat participant or something like that. We can keep the order the same in other cases except for when the user is typing to filter the / list, then @Terminal slash commands will be pushed down.
@roblourens roblourens enabled auto-merge (squash) May 25, 2024 19:29
@vscodenpa vscodenpa added this to the May 2024 milestone May 25, 2024
@roblourens roblourens merged commit f505673 into main May 26, 2024
7 checks passed
@roblourens roblourens deleted the roblou/positive-donkey branch May 26, 2024 00:00
andremmsilva pushed a commit to PIC1G55/vscodeG55 that referenced this pull request May 26, 2024
Fix microsoft/vscode-copilot-release#1245

This is a bit of a hack but has some advantanges over adding an 'order' field to the chat participant or something like that. We can keep the order the same in other cases except for when the user is typing to filter the / list, then @Terminal slash commands will be pushed down.
@microsoft microsoft locked and limited conversation to collaborators Jul 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

@terminal /explain is always sorted above @workspace /explain
3 participants