Skip to content

Update script to ping only on waiting-for-author label - #4812

Merged
Evan Mattson (moonbox3) merged 5 commits into
microsoft:mainfrom
moonbox3:update-ping-script
Mar 20, 2026
Merged

Evan Mattson (moonbox3) merged 5 commits into
microsoft:mainfrom
moonbox3:update-ping-script

Conversation

@moonbox3

@moonbox3 Evan Mattson (moonbox3) commented Mar 20, 2026 •

Copy link
Copy Markdown
Contributor

Motivation and Context

Update script to ping only on certain waiting-for-author label. Once pinged, a requested-info label is applied, so that we don't continuously re-ping.

Description

Update script to ping only on certain waiting for author label

Contribution Checklist

  • The code builds clean without any errors or warnings
  • The PR follows the Contribution Guidelines
  • All unit tests pass, and I have added new tests where possible
  • Is this a breaking change? If yes, add "[BREAKING]" prefix to the title of the PR.

Copilot AI review requested due to automatic review settings March 20, 2026 10:18

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the GitHub automation script that pings stale issues/PRs so it only considers items explicitly labeled as waiting on the author, reducing unnecessary pings across the repo.

Changes:

  • Filters the scan to only open issues/PRs labeled waiting-for-author.
  • Renames/splits label constants into TRIGGER_LABEL and PINGED_LABEL, and updates tests accordingly.
  • Adjusts test helpers to default mock issues to include the trigger label (reflecting the API pre-filter).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
.github/scripts/stale_issue_pr_ping.py Filters issue query by trigger label; updates labeling behavior/constants and messaging.
.github/tests/test_stale_issue_pr_ping.py Updates tests to use new label constants and aligns test data with pre-filter assumptions.

Comment thread .github/scripts/stale_issue_pr_ping.py Resolved
Comment thread .github/scripts/stale_issue_pr_ping.py Resolved
Comment thread .github/scripts/stale_issue_pr_ping.py Resolved
Comment thread .github/scripts/stale_issue_pr_ping.py Resolved
@moonbox3 Evan Mattson (moonbox3) changed the title Update script to ping only on certain waiting for author label Update script to ping only on waiting-for-author label Mar 20, 2026
@moonbox3
Evan Mattson (moonbox3) merged commit 8edcb28 into microsoft:main Mar 20, 2026
16 checks passed
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.

3 participants