Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
Expand Up @@ -38,4 +38,4 @@ Our automated bot will instantly check your game as soon as you submit this! To
4. **Is your game original?** The bot will check for plagiarism against all other games in the gallery.
5. **Did you include an image?** If you have an image, it must be in the [`/games/img` directory](https://github.com/hackclub/sprig/tree/main/games/img) and identically match your filename.

> Thanks for your PR!
> Thanks for your PR! Need help? Ask in #sprig on [Slack](https://hackclub.com/slack/) with your PR number, or tag `@LucasHT22` here.
4 changes: 2 additions & 2 deletions .github/scripts/auto-triage.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -156,7 +156,7 @@ try {
Auto triage could not complete validation for this submission:
> ${error.message}

Please check that submitted files can be accessed, or ask a maintainer for assistance.`,
Please check that submitted files can be accessed, or ask in #sprig on Slack (with your PR number) or tag \`@LucasHT22\` here for assistance.`,
});
process.exit(1);
}
Expand Down Expand Up @@ -701,7 +701,7 @@ ${links.join("\n")}

${checksSection}${warningLines.join("\n")}

${result.ok ? "Reviewers: please use the play link to playtest, then approve or request changes." : `@${pullRequest.user.login}: you can update your game right here ([edit file](${editUrl})) or push new commits. Once you do, these checks will rerun automatically! If you get stuck or have questions, ask in the #sprig channel on Slack or ping @LucasHT22 here.`}
${result.ok ? "Reviewers: please use the play link to playtest, then approve or request changes.\n\nAuthor: if your PR has been waiting for review for more than a few days, ask in #sprig on Slack (with your PR number) or tag `@LucasHT22` here." : `@${pullRequest.user.login}: you can update your game right here ([edit file](${editUrl})) or push new commits. Once you do, these checks will rerun automatically! If you get stuck or have questions, ask in #sprig on Slack (with your PR number) or tag \`@LucasHT22\` here.`}
`;
}

Expand Down
2 changes: 1 addition & 1 deletion .github/scripts/duplicate-detection.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ To keep reviews fair and fast for everyone, authors can only have one submission

- **Same game?** You do not need to open a new PR to update your game! Pushing commits or editing files in your existing PR updates it automatically. Usually your newer PR (**#${latestNumber}**) has your latest changes, but please check which PR has your best version, keep that one, and close the other.
- **Different games?** That is awesome, but our review team can only review one game per author at a time. Please choose which game you would like reviewed first, keep that PR open, and close the other. You can submit your next game once this one is merged!
- **Next steps:** If no action is taken, this older PR will close automatically in ${DUPLICATE_CLOSE_AFTER_DAYS} days so your newest PR can move forward. If you prefer to keep this older PR instead, please close #${latestNumber}. If you need any help, feel free to ask in the #sprig channel on Slack!`;
- **Next steps:** If no action is taken, this older PR will close automatically in ${DUPLICATE_CLOSE_AFTER_DAYS} days so your newest PR can move forward. If you prefer to keep this older PR instead, please close #${latestNumber}. If you need any help, feel free to ask in #sprig on Slack (with your PR number) or tag \`@LucasHT22\` here!`;
}

export function buildLatestWarning({ older }) {
Expand Down
8 changes: 4 additions & 4 deletions .github/scripts/stale-review-queue.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,7 @@ async function handleOlderDuplicate(pullRequest, labels) {
await commentOnce({
issueNumber: pullRequest.number,
marker: "<!-- sprig-duplicate-auto-close -->",
body: `Closing this older submission so your newer PR (#${latestNumber}) can move forward. If you would rather keep this game instead, close #${latestNumber} and ask a reviewer to reopen this PR.`,
body: `Closing this older submission so your newer PR (#${latestNumber}) can move forward. If you would rather keep this game instead, close #${latestNumber} and ask in #sprig on Slack (with your PR number) or tag \`@LucasHT22\` here to reopen this PR.`,
});
await githubRequest(token, "PATCH", `/repos/${owner}/${repo}/issues/${pullRequest.number}`, { state: "closed" });
closedDuplicateNumbers.add(pullRequest.number);
Expand Down Expand Up @@ -153,7 +153,7 @@ async function handleNeedsAuthor(pullRequest, labels) {
await commentOnce({
issueNumber: pullRequest.number,
marker: `<!-- sprig-auto-close-${cycle} -->`,
body: "Closing this submission for now since there has been no activity for 14 days. Whenever you want to return to it, push your fixes and ping a reviewer to reopen it. We would love to see your game in the gallery!",
body: "Closing this submission for now since there has been no activity for 14 days. Whenever you want to return to it, push your fixes and ask in #sprig on Slack (with your PR number) or tag `@LucasHT22` here to reopen it. We would love to see your game in the gallery!",
});
await githubRequest(token, "PATCH", `/repos/${owner}/${repo}/issues/${pullRequest.number}`, {
state: "closed",
Expand All @@ -166,7 +166,7 @@ async function handleNeedsAuthor(pullRequest, labels) {
await commentOnce({
issueNumber: pullRequest.number,
marker: `<!-- sprig-stale-reminder-${cycle} -->`,
body: "Just checking in! This submission has been waiting for updates for 7 days. Whenever you have time, push your fixes so we can get your game reviewed. If there is no response after 14 days, the PR will close automatically, but you can always ask a reviewer to reopen it whenever you are ready!",
body: "Just checking in! This submission has been waiting for updates for 7 days. Whenever you have time, push your fixes so we can get your game reviewed. If there is no response after 14 days, the PR will close automatically, but you can always ask in #sprig on Slack (with your PR number) or tag `@LucasHT22` here to reopen it whenever you are ready!",
});
} else if (hasLabel(labels, "Stale")) {
await removeLabel({ owner, repo, token, issueNumber: pullRequest.number, label: "Stale" });
Expand Down Expand Up @@ -227,7 +227,7 @@ async function handleUntouchedSubmission(pullRequest, labels) {
await commentOnce({
issueNumber: pullRequest.number,
marker: STALE_REMINDER_MARKER,
body: "This submission hasn't had any activity in over 30 days. Marking as Stale to get reviewer attention.",
body: "This submission hasn't had any activity in over 30 days. Marking as Stale to get reviewer attention. If your submission is ready for review, ask in #sprig on Slack (with your PR number) or tag `@LucasHT22` here.",
});
}
}
Expand Down
Loading