ci: repo governance workflows and assignment policy#713
Open
andreahlert wants to merge 3 commits intoapache:mainfrom
Open
ci: repo governance workflows and assignment policy#713andreahlert wants to merge 3 commits intoapache:mainfrom
andreahlert wants to merge 3 commits intoapache:mainfrom
Conversation
Weekly cron job (Mondays 09:00 UTC) that enforces the PR lifecycle policy defined in apache#690: - 14 days after review with no author response: label pr/stale and convert to draft - 90 days with no activity: close with comment - Skip PRs with lifecycle/frozen, pr/do-not-merge, or from dependabot Closes apache#695 Signed-off-by: André Ahlert <andre@aex.partners>
andreahlert
added
kind/improvement
Collaborator
Author
|
@skrawcz @elijahbenizzy can you take a look? this is the automated stale PR policy we discussed in #690. runs weekly, marks inactive PRs after review as draft, closes after 90 days. skips anything with lifecycle/frozen or pr/do-not-merge |
- New issues automatically get `status/needs-triage` - PRs get `area/*` labels based on changed files via actions/labeler - PRs get `pr/needs-rebase` when they have merge conflicts (auto-removed when resolved) Labeler config maps repo paths to area labels: core, ui, storage, streaming, hooks, tracking, integrations, visualization, website, ci, examples, typing. Closes apache#696 Signed-off-by: André Ahlert <andre@aex.partners>
andreahlert
changed the title
Documents the assignment policy in CONTRIBUTING.rst: - Assignee = actively working - 14 days without activity: warning comment - 21 days total: unassign + add help wanted - lifecycle/frozen issues are exempt Adds weekly workflow (Wednesdays 09:00 UTC) to enforce it automatically. Closes apache#709 Signed-off-by: André Ahlert <andre@aex.partners>
andreahlert
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Implements the remaining sub-issues from #690 (repo governance):
Stale PR lifecycle (#695)
pr/stale+ convert to draftlifecycle/frozen,pr/do-not-merge, and dependabotAuto-labeling (#696)
status/needs-triageautomaticallyarea/*labels based on changed files viaactions/labelerpr/needs-rebasewhen they have merge conflicts (auto-removed when resolved)Issue assignment policy (#709)
help wantedlifecycle/frozenissues are exemptFiles
.github/workflows/stale-prs.yml- stale PR lifecycle.github/workflows/auto-label.yml- auto-label issues and PRs.github/workflows/stale-assignments.yml- stale assignment enforcement.github/labeler.yml- path-to-label mapping configCONTRIBUTING.rst- assignment policy documentationTest plan
workflow_dispatchworkflow_dispatchstatus/needs-triageis appliedburr/core/and verifyarea/coreis appliedCloses #695
Closes #696
Closes #709