Skip to content

Commit

Permalink
Fix review comments
Browse files Browse the repository at this point in the history
  • Loading branch information
X-Guardian committed Jun 29, 2020
1 parent 620fd5c commit 79c3530
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 6 deletions.
6 changes: 3 additions & 3 deletions GitHubConfiguration.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -244,7 +244,7 @@ function Set-GitHubConfiguration
}
}

if (-not $PSCmdlet.ShouldProcess('GitHubConfiguration',' Save'))
if (-not $PSCmdlet.ShouldProcess('GitHubConfiguration','Save'))
{
return
}
Expand Down Expand Up @@ -344,7 +344,7 @@ function Save-GitHubConfiguration
[string] $Path
)

if (-not $PSCmdlet.ShouldProcess('GitHub Configuration','Save'))
if (-not $PSCmdlet.ShouldProcess('GitHub Configuration', 'Save'))
{
return
}
Expand Down Expand Up @@ -514,7 +514,7 @@ function Reset-GitHubConfiguration

Set-TelemetryEvent -EventName Reset-GitHubConfiguration

if (-not $PSCmdlet.ShouldProcess('GitHub Configuration','Reset'))
if (-not $PSCmdlet.ShouldProcess('GitHub Configuration', 'Reset'))
{
return
}
Expand Down
2 changes: 1 addition & 1 deletion GitHubIssueComments.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -550,7 +550,7 @@ filter Set-GitHubIssueComment
'body' = $Body
}

if (-not $PSCmdlet.ShouldProcess($Issue, 'Create GitHub Issue Comment'))
if (-not $PSCmdlet.ShouldProcess($Comment, 'Update GitHub Issue Comment'))
{
return
}
Expand Down
2 changes: 1 addition & 1 deletion GitHubProjectColumns.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -477,7 +477,7 @@ filter Move-GitHubProjectColumn
'position' = $Position
}

if (-not $PSCmdlet.ShouldProcess($ColumnName, 'Move GitHub Project Column'))
if (-not $PSCmdlet.ShouldProcess($Column, 'Move GitHub Project Column'))
{
return
}
Expand Down
4 changes: 3 additions & 1 deletion GitHubPullRequests.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -391,6 +391,7 @@ filter New-GitHubPullRequest
if ($PSBoundParameters.ContainsKey('Title'))
{
$description = "Creating pull request $Title in $RepositoryName"
$shouldProcessAction = "Create GitHub Pull Request $Title"
$postBody['title'] = $Title

# Body may be whitespace, although this might not be useful
Expand All @@ -402,6 +403,7 @@ filter New-GitHubPullRequest
else
{
$description = "Creating pull request for issue $Issue in $RepositoryName"
$shouldProcessAction = "Create GitHub Pull Request for Issue $Issue"
$postBody['issue'] = $Issue
}

Expand All @@ -416,7 +418,7 @@ filter New-GitHubPullRequest
$acceptHeader = 'application/vnd.github.shadow-cat-preview+json'
}

if (-not $PSCmdlet.ShouldProcess($Title, 'Create GitHub Pull Request'))
if (-not $PSCmdlet.ShouldProcess($RepositoryName, $shouldProcessAction))
{
return
}
Expand Down

0 comments on commit 79c3530

Please sign in to comment.