Skip to content

fix bug with weekly status bot label filtering#12301

Merged
mekarpeles merged 1 commit intomasterfrom
fix/weekly-status-bug
Apr 6, 2026
Merged

fix bug with weekly status bot label filtering#12301
mekarpeles merged 1 commit intomasterfrom
fix/weekly-status-bug

Conversation

@RayBB
Copy link
Copy Markdown
Collaborator

@RayBB RayBB commented Apr 6, 2026

Closes #12302

Technical

Testing

Screenshot

Stakeholders

@mekarpeles
Copy link
Copy Markdown
Member

mekarpeles commented Apr 6, 2026

Thanks for the fix @RayBB! 🤖 Copilot has been assigned for an initial review.

The linked issue #12302 hasn't been triaged yet — triage happens on Mondays and Fridays.

@mekarpeles is assigned to your PR and currently has:

  • 13 PRs of equal or higher priority to review first

For future PRs, please follow the PR template and provide a description. Here's a draft based on the diff and linked issue:

The weekly status report script was filtering PRs using the label Needs: Response when counting submitter input items, but the correct label is Needs: Submitter Input. This caused the "Submitter Input for our PRs" count to display incorrectly. The fix is a 1-line label name correction in scripts/gh_scripts/weekly_status_report.mjs.

PR triage checklist (maintainers / Pam)

Note

This comment was automatically generated by Pam, Open Library's Project AI Manager, on behalf of @mekarpeles.

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

Fixes the weekly status bot’s “Submitter Input for our PRs” section by updating the label it counts to match the intended GitHub label.

Changes:

  • Update prepareSubmitterInput to count PRs labeled Needs: Submitter Input (instead of Needs: Response).

@mekarpeles mekarpeles merged commit 4f1a276 into master Apr 6, 2026
11 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.

Weekly status bot isn't searching for needs submitter input correctly

3 participants