Skip to content

Commit

Permalink
Fix issue where inputs were thought to be required.
Browse files Browse the repository at this point in the history
  • Loading branch information
IEvangelist committed Feb 2, 2024
1 parent cf9dcc5 commit 1aa14ce
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ internal static class CoreServiceExtensions
}

// Consumers can opt-out of this feature.
if (core.GetBoolInput("include-updated-note") is false)
if (core.GetBoolInput("include-updated-note", new() { Required = false }) is false)
{
return finalResult;
}
Expand Down
2 changes: 1 addition & 1 deletion src/ProfanityFilter.Action/ProfanityProcessor.Issue.cs
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ private async Task<FiltrationResult> HandleIssueAsync(

await client.UpdateIssueAsync((int)issueNumber, issueUpdate);

if (core.GetBoolInput("include-confused-reaction"))
if (core.GetBoolInput("include-confused-reaction", new() { Required = false }))
{
await client.AddReactionAsync(issueNumber, ReactionContent.Confused);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ private async Task<FiltrationResult> HandleIssueCommentAsync(

await client.UpdateIssueCommentAsync(issueCommentId, finalBodyUpdate);

if (core.GetBoolInput("include-confused-reaction"))
if (core.GetBoolInput("include-confused-reaction", new() { Required = false }))
{
var issueNumber = (int)_context!.Payload!.Issue!.Number;
await client.AddReactionAsync(issueNumber, ReactionContent.Confused);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ private async Task<FiltrationResult> HandlePullRequestAsync(
await client.UpdatePullRequestAsync(
(int)pullRequestNumber, pullRequestUpdate, label?.Name);

if (core.GetBoolInput("include-confused-reaction"))
if (core.GetBoolInput("include-confused-reaction", new() { Required = false }))
{
await client.AddReactionAsync(
pullRequestNumber, ReactionContent.Confused);
Expand Down

0 comments on commit 1aa14ce

Please sign in to comment.