Skip to content

Commit 8ed9fd9

Browse files
use foreach loop
1 parent bb78571 commit 8ed9fd9

File tree

1 file changed

+17
-17
lines changed

1 file changed

+17
-17
lines changed

eng/scripts/Set-SkipJobVariables.ps1

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -28,25 +28,25 @@ $changedFilesScriptPath = "$RepoRoot/eng/common/scripts/get-changedfiles.ps1"
2828
$changedFiles = & $changedFilesScriptPath -DiffFilterType ''
2929

3030
if ($changedFiles) {
31-
$changedFiles | ForEach-Object {
32-
Write-Host "Checking file: $_"
33-
$fileName = [System.IO.Path]::GetFileName($_)
34-
$isSkipFile = $skipFiles -contains $fileName
35-
36-
$isInSkipDirectory = $false
37-
foreach ($dir in $skipDirectories) {
38-
if ($_.StartsWith($dir, [System.StringComparison]::OrdinalIgnoreCase)) {
39-
$isInSkipDirectory = $true
40-
break
31+
foreach ($file in $changedFiles) {
32+
Write-Host "Checking file: $file"
33+
$fileName = [System.IO.Path]::GetFileName($file)
34+
$isSkipFile = $skipFiles -contains $fileName
35+
36+
$isInSkipDirectory = $false
37+
foreach ($dir in $skipDirectories) {
38+
if ($file.StartsWith($dir, [System.StringComparison]::OrdinalIgnoreCase)) {
39+
$isInSkipDirectory = $true
40+
break
41+
}
4142
}
42-
}
4343

44-
Write-Host " -> File: $fileName, IsSkipFile: $isSkipFile, IsInSkipDir: $isInSkipDirectory"
45-
if (-not $isSkipFile -and -not $isInSkipDirectory) {
46-
$skipBuildAndTest = $false
47-
Write-Host " -> File: $_ requires build and test!"
48-
return
49-
}
44+
Write-Host " -> File: $fileName, IsSkipFile: $isSkipFile, IsInSkipDir: $isInSkipDirectory"
45+
if (-not $isSkipFile -and -not $isInSkipDirectory) {
46+
$skipBuildAndTest = $false
47+
Write-Host " -> File: $file requires build and test!"
48+
return
49+
}
5050
}
5151
}
5252

0 commit comments

Comments
 (0)