Reviewing contributions
-The nixpkgs projects receives a fairly high number of contributions via github pull-requests. Reviewing and approving these is an important task and a way to contribute to the project.
+The nixpkgs projects receives a fairly high number of contributions via GitHub pull-requests. Reviewing and approving these is an important task and a way to contribute to the project.The high change rate of nixpkgs make any pull request that is open for long enough subject to conflicts that will require extra work from the submitter or the merger. Reviewing pull requests in a timely manner and being responsive to the comments is the key to avoid these. Github provides sort filters that can be used to see the most recently and the least recently updated pull-requests.When reviewing a pull request, please always be nice and polite. Controversial changes can lead to controversial opinions, but it is important to respect every community members and their work.
-Github provides emoji, they are a simple and quick way to provide feedback to pull-requests or any comments. The thumb-down emoji should be used with care and if possible accompanied with some explanations so the submitter has directions to improve his contribution.
+GitHub provides reactions, they are a simple and quick way to provide feedback to pull-requests or any comments. The thumb-down reaction should be used with care and if possible accompanied with some explanations so the submitter has directions to improve his contribution.Pull-requests reviews should include a list of what has been reviewed in a comment, so other reviewers and mergers can know the state of the review.
@@ -35,7 +35,7 @@
Ensure that the commit text is fitting the guidelines.Ensure that the package maintainers are notified.
- mention-bot usually notify github users based on the submitted changes, but it can happen that it misses some of the package maintainers.
+ mention-bot usually notify GitHub users based on the submitted changes, but it can happen that it misses some of the package maintainers.Ensure that the meta field contains correct information.
@@ -52,8 +52,8 @@
$ git remote add channels https://github.com/NixOS/nixpkgs-channels.git
$ git fetch channels nixos-unstable
-$ git fetch origin pull/PRNUMBER/head:PRNUMBER
-$ git rebase --onto nixos-unstable PRNUMBER
+$ git fetch origin pull/PRNUMBER/head
+$ git rebase --onto nixos-unstable BASEBRANCH FETCH_HEAD
@@ -63,7 +63,7 @@ $ git rebase --onto nixos-unstable PRNUMBER Fetching the nixos-unstable branch.
- Fetching the pull-request changes, PRNUMBER is the number at the end of the pull-request title.
+ Fetching the pull-request changes, PRNUMBER is the number at the end of the pull-request title and BASEBRANCH the base branch of the pull-request.Rebasing the pull-request changes to the nixos-unstable branch.
@@ -124,7 +124,7 @@ $ nix-shell -p nox --run "nox-review -k pr PRNUMBER"
Ensure the package source.Mirrors urls should be used when available.
- The most appropriate function should be used (e.g. packages from github should use fetchFromGithub).
+ The most appropriate function should be used (e.g. packages from GitHub should use fetchFromGitHub).Building the package locally.
@@ -170,7 +170,7 @@ $ nix-shell -p nox --run "nox-review -k pr PRNUMBER"
Ensure that the module maintainers are notified.
- Mention-bot notify github users based on the submitted changes, but it can happen that it miss some of the package maintainers.
+ Mention-bot notify GitHub users based on the submitted changes, but it can happen that it miss some of the package maintainers.Ensure that the module tests, if any, are succeeding.
@@ -278,7 +278,13 @@ The main reviewers for a topic can be hard to find as there is no list, but chec
TODO: add the procedure to request merging rights.
+
In a case a contributor leaves definitively the Nix community, he should create an issue or notify the mailing list with references of packages and modules he maintains so the maintainership can be taken over by other contributors.