Skip to content

Commit 297f123

Browse files
authored
Merge pull request #32 from MattSturgeon/main
Clarify what commit access is for in retirement PRs
2 parents 9f71c8a + 9734d2a commit 297f123

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

scripts/retire.sh

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -167,6 +167,10 @@ for login in *; do
167167
echo "This is an automated PR to retire @$login as a Nixpkgs committers due to not using their commit access for 1 year."
168168
echo ""
169169
echo "Make a comment with your motivation to keep commit access, otherwise this PR will be merged and implemented in 1 month."
170+
echo ""
171+
echo "> [!NOTE]"
172+
echo -n "> Commit access is not required for most forms of contributing, including being a maintainer and reviewing PRs."
173+
echo ' It is only needed for things that require `write` permissions to Nixpkgs, such as merging PRs.'
170174
} | effect gh api \
171175
--method POST \
172176
/repos/"$ORG"/"$MEMBER_REPO"/pulls \

0 commit comments

Comments
 (0)