Skip to content

Commit

Permalink
Remove the needless loop and, now defunct, comment
Browse files Browse the repository at this point in the history
  • Loading branch information
JimSuplizio authored and azure-sdk committed Feb 21, 2025
1 parent 70741f3 commit 6a073f6
Showing 1 changed file with 1 addition and 8 deletions.
9 changes: 1 addition & 8 deletions eng/common/scripts/Package-Properties.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -185,14 +185,7 @@ function Get-PrPkgProperties([string]$InputDiffJson) {
$targetedFiles += $diff.DeletedFiles
}

# The exclude paths and the targeted files paths aren't full OS paths, they're
# GitHub paths meaning they're relative to the repo root and slashes are forward
# slashes "/". The ExcludePaths need to have a trailing slash added in order
# correctly test for string matches without overmatching. For example, if a pr
# had files sdk/foo/file1 and sdk/foobar/file2 with the exclude of anything in
# sdk/foo, it should only exclude things under sdk/foo. The TrimEnd is just in
# case one of the paths ends with a slash, it doesn't add a second one.
$excludePaths = $diff.ExcludePaths | ForEach-Object { $_ }
$excludePaths = $diff.ExcludePaths

$additionalValidationPackages = @()
$lookup = @{}
Expand Down

0 comments on commit 6a073f6

Please sign in to comment.