Skip to content

Commit 958ddab

Browse files
Validate ReleaseType input to ensure it is provided and valid, updating related variables for clarity
1 parent d405e6b commit 958ddab

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

scripts/helpers/Publish-PSModule.ps1

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@
108108
Set-GitHubLogGroup 'Calculate release type' {
109109
$prereleaseName = $prHeadRef -replace '[^a-zA-Z0-9]'
110110

111-
# Validate ReleaseType - must be provided by Get-PSModuleSettings
111+
# Validate ReleaseType - fail if not provided or invalid to catch configuration errors
112112
$validReleaseTypes = @('Release', 'Prerelease', 'Cleanup', 'None')
113113
if ([string]::IsNullOrWhiteSpace($releaseType)) {
114114
Write-Error "ReleaseType input is required. Valid values are: $($validReleaseTypes -join ', ')"
@@ -121,7 +121,7 @@
121121

122122
$createRelease = $releaseType -eq 'Release'
123123
$createPrerelease = $releaseType -eq 'Prerelease'
124-
$closedPullRequest = $releaseType -eq 'Cleanup'
124+
$isCleanupMode = $releaseType -eq 'Cleanup'
125125

126126
if ($releaseType -eq 'None') {
127127
Write-Output 'ReleaseType is None. Skipping release creation.'
@@ -147,7 +147,7 @@
147147
Write-Output "Create a major release: [$majorRelease]"
148148
Write-Output "Create a minor release: [$minorRelease]"
149149
Write-Output "Create a patch release: [$patchRelease]"
150-
Write-Output "Cleanup prereleases: [$closedPullRequest]"
150+
Write-Output "ReleaseType is Cleanup: [$isCleanupMode]"
151151
Write-Output '-------------------------------------------------'
152152
}
153153

@@ -450,7 +450,7 @@
450450
$prereleasesToCleanup | Select-Object -Property name, publishedAt, isPrerelease, isLatest | Format-Table | Out-String
451451
}
452452

453-
if ((($closedPullRequest -or $createRelease) -and $autoCleanup) -or $whatIf) {
453+
if ((($isCleanupMode -or $createRelease) -and $autoCleanup) -or $whatIf) {
454454
Set-GitHubLogGroup "Cleanup prereleases for [$prereleaseName]" {
455455
foreach ($rel in $prereleasesToCleanup) {
456456
$relTagName = $rel.tagName

0 commit comments

Comments
 (0)