summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorWolfgang Walther <walther@technowledgy.de>2025-07-18 09:23:17 +0200
committergithub-actions[bot] <github-actions[bot]@users.noreply.github.com>2026-01-18 21:05:20 +0000
commit6d0a1f3ee02082555cd4a8690059bd03c1a063aa (patch)
treee65d7b7a24b546c5868865ab35e529ace52133a1
parent4d2b249a94fd32c0d09046c37e80b83a1c5cb1fc (diff)
.github/PULL_REQUEST_TEMPLATE: remove +1 footer
The footer has the following problems: - It's not consistently added in all PRs, for example backports or those where the template is not used. Inconsistency in a voting skews the results. - It's arguably not effective, because people use reactions naturally to interact with PRs anyway. As long as there are multiple different reaction types, what does it mean that one PR has 20 :+1:, but another has 22 :tada:? The latter won't show up in the ranking for :+1:, so at the minimum confusing. Same for :heart:. - It's not useful to determine priority as in "higher number of votes should be merged". A PR's quality should be determined independently of votes - when it's good, it will be merged. The currently most upvoted PR has serious issues, so this invites drive-by questions like "Why isn't this merged, given the number of upvotes?". (cherry picked from commit c89c90dfa137316f3cf662783b4795d8b9897237)
-rw-r--r--.github/PULL_REQUEST_TEMPLATE.md7
1 files changed, 0 insertions, 7 deletions
diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md
index 85561029c86a..45a55eb45dce 100644
--- a/.github/PULL_REQUEST_TEMPLATE.md
+++ b/.github/PULL_REQUEST_TEMPLATE.md
@@ -38,10 +38,3 @@ For new packages please briefly describe the package or provide a link to its ho
[nixos/tests]: https://github.com/NixOS/nixpkgs/blob/master/nixos/tests
[pkgs/README.md]: https://github.com/NixOS/nixpkgs/blob/master/pkgs/README.md
[pkgs/test]: https://github.com/NixOS/nixpkgs/blob/master/pkgs/test
-
----
-
-Add a :+1: [reaction] to [pull requests you find important].
-
-[reaction]: https://github.blog/2016-03-10-add-reactions-to-pull-requests-issues-and-comments/
-[pull requests you find important]: https://github.com/NixOS/nixpkgs/pulls?q=is%3Aopen+sort%3Areactions-%2B1-desc