Skip to content

Commit

Permalink
Use of dotnet nuget push in favor of nuget push
Browse files Browse the repository at this point in the history
As nuget.exe is not cross platform(*), `dotnet` should be used instead.

*It kind of is, but this explains it quite well: NuGet/setup-nuget#168 (comment)
  • Loading branch information
seveneleven committed Jan 13, 2025
1 parent e12f3e9 commit 5425cb6
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .build/BuildToolkit.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -477,7 +477,7 @@ function Invoke-Publish([string]$NugetApiKey, [string]$PackageTarget, [string]$S

foreach ($package in $packages) {
Write-Host "Pushing package $package"
nuget push $package -ApiKey $NugetApiKey -NoSymbols -SkipDuplicate -Source $PackageTarget
& dotnet nuget push $package --api-key $NugetApiKey --no-symbols --skip-duplicate --source $PackageTarget
Invoke-ExitCodeCheck $LastExitCode;
}

Expand All @@ -489,7 +489,7 @@ function Invoke-Publish([string]$NugetApiKey, [string]$PackageTarget, [string]$S

foreach ($symbolPackage in $symbolPackages) {
Write-Host "Pushing symbol (snupkg) $symbolPackage"
nuget push $symbolPackage -ApiKey $NugetApiKey -SkipDuplicate -Source $SymbolTarget
& dotnet nuget push $symbolPackage --api-key $NugetApiKey --skip-duplicate --source $SymbolTarget
Invoke-ExitCodeCheck $LastExitCode;
}
}
Expand Down

0 comments on commit 5425cb6

Please sign in to comment.