豆豆友情提示:这是一个非官方 GitHub 代理镜像,主要用于网络测试或访问加速。请勿在此进行登录、注册或处理任何敏感信息。进行这些操作请务必访问官方网站 github.com。 Raw 内容也通过此代理提供。
Skip to content

pass arguments directly instead of double-interpolating them#13171

Merged
brettfo merged 1 commit intomainfrom
dev/brettfo/nuget-script-args-escape
Sep 24, 2025
Merged

pass arguments directly instead of double-interpolating them#13171
brettfo merged 1 commit intomainfrom
dev/brettfo/nuget-script-args-escape

Conversation

@brettfo
Copy link
Copy Markdown
Contributor

@brettfo brettfo commented Sep 24, 2025

Don't use string interpolation to pass dynamic arguments to the updater because certain PowerShell special characters can cause issues like $, etc. Instead directly pass the string array to Start-Process.

@brettfo brettfo requested a review from a team as a code owner September 24, 2025 19:38
@github-actions github-actions bot added the L: dotnet:nuget NuGet packages via nuget or dotnet label Sep 24, 2025
@brettfo brettfo merged commit bbb808b into main Sep 24, 2025
118 of 120 checks passed
@brettfo brettfo deleted the dev/brettfo/nuget-script-args-escape branch September 24, 2025 21:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

L: dotnet:nuget NuGet packages via nuget or dotnet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants