Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -109,14 +109,14 @@ which should be merged by the Nixpkgs commit delegators to reconcile the mismatc
### Semi-automatic retirement

Every day, [a GitHub Action workflow](./.github/workflows/retire.yml) runs
to check if any Nixpkgs committers have not used their commit access within the last year,
to check if any Nixpkgs committers have not used their commit access within the last six months,
in which case an automated PR is created to remove them from the member list.

The PR will ping the user and inform them that it will by default be merged and implemented in one month.
If the PR is still open one month later,
an automated comment will be posted with the next steps for the Nixpkgs commit delegators.

If the PR is closed, retirement is delayed by another year.
If the PR is closed, retirement is delayed by another six months.

### Setup

Expand Down
2 changes: 1 addition & 1 deletion scripts/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ This script has external effects and as such needs a bit more care when testing.

### Setup (important!)

To avoid other users getting pings, ensure that the `members-test` directory contains only simulated new users and your own user (simulated to have been added over a year ago), then commit and push it for testing:
To avoid other users getting pings, ensure that the `members-test` directory contains only simulated new users and your own user (simulated to have been added over six months ago), then commit and push it for testing:

```bash
me=$(gh api /user --jq .login)
Expand Down
8 changes: 4 additions & 4 deletions scripts/retire-config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,19 +5,19 @@ export default {
noticeCutoffSpec: 'yesterday 1 month ago',

// Skip users whose retirement PR was closed more recently than this.
closeCutoffSpec: '1 year ago',
closeCutoffSpec: '6 months ago',

// Require repeated inactivity observations over this period before opening a PR.
confirmCutoffSpec: '7 days ago',

// Skip people who received their commit bit more recently than this.
newCutoffSpec: '1 year ago',
newCutoffSpec: '6 months ago',

// People are active if they merged a PR more recently than this.
activityCutoffSpec: '1 year ago',
activityCutoffSpec: '6 months ago',

// Trust cached merges newer than this without querying GitHub again.
// The gap until the activity cutoff gives about a month of daily queries
// agreeing before a retirement PR can be opened (see issue #100).
queryCutoffSpec: '11 months ago',
queryCutoffSpec: '5 months ago',
};
Loading