Skip to content

Commit

Permalink
NextMajor fixes: Sorting is broken in Get-BcContainerAppInfo (#3367)
Browse files Browse the repository at this point in the history
In BCApps we use Get-BcContainerAppInfo to get a sorted list of apps so
we can uninstall them in the right order. In the latest artifacts they
are not sorted in the right order though

See failures in: 
https://github.com/microsoft/BCApps/actions/runs/8015836387
or 
https://github.com/aholstrup1/BCApps/actions/runs/8023595403

With this change, the build passes again
https://github.com/aholstrup1/BCApps/actions/runs/8023619694
  • Loading branch information
aholstrup1 authored Feb 23, 2024
1 parent 714ff2d commit 174fa46
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion AppHandling/Get-NavContainerAppInfo.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ try {

function AddDependency { Param($dependency)
#Write-Host "Add Dependency $($dependency.Name) $($dependency.Version)"
$dependentApp = $apps | Where-Object { $_.AppId -eq $dependency.AppId }
$dependentApp = $apps | Where-Object { "$($_.AppId)" -eq "$($dependency.AppId)" }
if ($dependentApp) {
@($dependentApp) | ForEach-Object { AddAnApp -AnApp $_ }
}
Expand Down

0 comments on commit 174fa46

Please sign in to comment.