Skip to content

fix: triage misplaced game submissions and handle manual Submission label - #4259

Open
SSoggyTacoMan wants to merge 2 commits into
hackclub:mainfrom
SSoggyTacoMan:fix/misplaced-submission-triage
Open

SSoggyTacoMan wants to merge 2 commits into
hackclub:mainfrom
SSoggyTacoMan:fix/misplaced-submission-triage

Conversation

@SSoggyTacoMan

Copy link
Copy Markdown
Contributor

When submitters place game files outside the games/ directory (such as in the repository root or uppercase Games/), the auto-triage workflow previously skipped execution with 'Not a submission PR', because it strictly checked filename.startsWith('games/') and the new PR lacked the Submission label. The bot never commented to inform the submitter where the file belonged.

Furthermore, when a maintainer manually added the Submission label, the workflow did not trigger because labeled was not included in the workflow event types.

This change:

  • Adds labeled to pull_request_target types in auto-triage.yml, ignoring events where the added label is not Submission.
  • Checks games/ case-insensitively.
  • Recognizes game submissions by PR body template so misplaced files trigger triage, report the placement error, and provide the edit link.

@vercel

vercel Bot commented Sep 21, 2026

Copy link
Copy Markdown

@SSoggyTacoMan is attempting to deploy a commit to the Hack Club Team on Vercel.

A member of the Team first needs to authorize it.

This branch has not been deployed

No deployments
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.

1 participant