Skip to content

Improve the daily.yml workflow #8737

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Sep 15, 2022

Conversation

AlexWaygood
Copy link
Member

  1. Make it easier to run the workflow from a typeshed fork using the workflow_dispatch trigger. This is something I do sporadically if a mypy release is coming up, so we don't get blindsided by loads of very disruptive stubtest changes when the release is cut. Making this change means I don't have to manually edit the workflow to run it from my private fork, which is a pain :)
  2. Auto-add the "help wanted" label to issues created by the bot when the daily test fails. @kkirsche brought up in Evaluate ways to determine accuracy and reduce incomplete type stubs #8481 (comment) that it's sometimes pretty difficult for contributors to know which typeshed issues we'd welcome help on, and which issues aren't really actionable, or are pretty out of date. I think that's a very reasonably criticism, and I've been trying to improve on that by using the "help wanted" label more often, to indicate issues where a maintainer has confirmed that a PR to fix the issue would be welcome. Pretty much any issue auto-created by the bot due to a stubtest failure can be considered "auto-triaged", so let's automatically add the label to those issues.

(github.event_name is documented here. Using the GitHub API to create issues with specific labels is documented here.)

@hauntsaninja hauntsaninja merged commit 3e71efb into python:master Sep 15, 2022
@AlexWaygood AlexWaygood deleted the daily-workflow branch September 15, 2022 08:44
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