Skip to content

fix: runCommand type #246198

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
Merged
meganrogge merged 2 commits into from
Apr 14, 2025
Merged

fix: runCommand type #246198

meganrogge merged 2 commits into from
Apr 14, 2025

Conversation

@futurist
Copy link
Contributor

@futurist futurist commented Apr 10, 2025

  1. fix a typo
  2. fix a type mismatch with implementation

@roblourens roblourens assigned Tyriar and meganrogge and unassigned roblourens Apr 10, 2025
meganrogge
meganrogge approved these changes Apr 14, 2025
View reviewed changes
Copy link
Contributor

@meganrogge meganrogge left a comment

Choose a reason for hiding this comment

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

Thanks

@meganrogge meganrogge enabled auto-merge (squash) April 14, 2025 14:01
@vs-code-engineering vs-code-engineering bot added this to the April 2025 milestone Apr 14, 2025
@meganrogge meganrogge merged commit 6717967 into microsoft:main Apr 14, 2025
7 checks passed
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators May 29, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@jrieken jrieken jrieken approved these changes

@meganrogge meganrogge meganrogge approved these changes

Assignees

@Tyriar Tyriar

@meganrogge meganrogge

Labels

None yet

Projects

None yet

Milestone

April 2025

Development

Successfully merging this pull request may close these issues.

5 participants

@futurist @jrieken @meganrogge @roblourens @Tyriar