Skip to content
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

feat(update-issues): Process PRs as well as issues #25

Merged
merged 3 commits into from
Jul 24, 2023

Conversation

joeyparrish
Copy link
Member

The Shaka Player maintainers list has discussed in the past the idea of processing PRs as well as issues with our automated tools. I think it is particularly useful for managing stale PRs with the "waiting" tag.

Changes to processing:

  • Do not skip PRs at a high level
  • Archive PRs with the same rules used for issues
  • Unarchive PRs with the same rules used for issues, except that PRs won't be automatically reopened
  • Close stale PRs with the same rules as issues, but with a different message (since PRs can't be automatically reopened in many cases)
  • Clean up "waiting" tags from closed PRs (same as issues)
  • Clean up "waiting" tags from PRs after a new comment (same as issues)

PRs are excluded from these issue processing features:

  • Reopen issue by specially-formatted OP comment, since PRs can't be automatically reopened in many cases
  • Any rules for issues with the "question" tag, which wouldn't make sense for PRs
  • Automatic milestone management

Changes to tests:

  • Set defaults for isPR, merged in mocks
  • Maintain basic state in mock methods (fixes several pre-existing test failures)
  • Add PR scenarios to test cases
  • Add missing test cases for pre-existing reopen features

The Shaka Player maintainers list has discussed in the past the idea
of processing PRs as well as issues with our automated tools.  I think
it is particularly useful for managing stale PRs with the "waiting"
tag.

Changes to processing:
 - Do not skip PRs at a high level
 - Archive PRs with the same rules used for issues
 - Unarchive PRs with the same rules used for issues, except that PRs
   won't be automatically reopened
 - Close stale PRs with the same rules as issues, but with a different
   message (since PRs can't be automatically reopened in many cases)
 - Clean up "waiting" tags from closed PRs (same as issues)
 - Clean up "waiting" tags from PRs after a new comment (same as
   issues)

PRs are excluded from these issue processing features:
 - Reopen issue by specially-formatted OP comment, since PRs can't be
   automatically reopened in many cases
 - Any rules for issues with the "question" tag, which wouldn't make
   sense for PRs
 - Automatic milestone management

Changes to tests:
 - Set defaults for isPR, merged in mocks
 - Maintain basic state in mock methods (fixes several pre-existing
   test failures)
 - Add PR scenarios to test cases
 - Add missing test cases for pre-existing reopen features
update-issues/main.js Show resolved Hide resolved
update-issues/main.js Outdated Show resolved Hide resolved
@joeyparrish
Copy link
Member Author

@theodab: ping!

@joeyparrish joeyparrish merged commit 24ed1b2 into shaka-project:main Jul 24, 2023
3 checks passed
@joeyparrish joeyparrish deleted the process-prs branch July 25, 2023 19:41
@github-actions github-actions bot added the status: archived Archived and locked; will not be updated label Sep 22, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: archived Archived and locked; will not be updated
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants