Skip to content

chore: fix new contributor pr message failing due to required, unused parameters#6675

Merged
CommanderStorm merged 4 commits intolouislam:masterfrom
CommanderStorm:CommanderStorm-patch-4
Jan 11, 2026
Merged

chore: fix new contributor pr message failing due to required, unused parameters#6675
CommanderStorm merged 4 commits intolouislam:masterfrom
CommanderStorm:CommanderStorm-patch-4

Conversation

@CommanderStorm
Copy link
Copy Markdown
Collaborator

No description provided.

Removed comments explaining the workflow purpose.
Updated PR message to include a required parameter note.
Copilot AI review requested due to automatic review settings January 11, 2026 13:44
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes a failing GitHub workflow that welcomes new contributors. The workflow was failing because the actions/first-interaction action requires specific parameters that were either missing or incorrectly named.

Changes:

  • Added the required issue_message parameter with a placeholder message
  • Corrected the parameter name from pr-message to pr_message (kebab-case to snake_case)

with:
repo-token: ${{ secrets.GITHUB_TOKEN }}
pr-message: |
issue_message: "This is a required parameter, but we don't run on this event as we consider it a bit spammy.."
Copy link

Copilot AI Jan 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are two periods at the end of this message instead of one. The message should end with a single period.

Suggested change
issue_message: "This is a required parameter, but we don't run on this event as we consider it a bit spammy.."
issue_message: "This is a required parameter, but we don't run on this event as we consider it a bit spammy."

Copilot uses AI. Check for mistakes.
@CommanderStorm CommanderStorm merged commit b02d679 into louislam:master Jan 11, 2026
24 checks passed
@CommanderStorm CommanderStorm added this to the 2.1.0 milestone Jan 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants