Merge pull request #175074 from marsam/1password-fix-hashes
[NixPkgs.git] / .github / PULL_REQUEST_TEMPLATE.md
blob397e5ff5d175007354826f110ecf9b0794549700
1 ###### Description of changes
3 <!--
4 For package updates please link to a changelog or describe changes, this helps your fellow maintainers discover breaking updates.
5 For new packages please briefly describe the package or provide a link to its homepage.
6 -->
8 ###### Things done
10 <!-- Please check what applies. Note that these are not hard requirements but merely serve as information for reviewers. -->
12 - Built on platform(s)
13   - [ ] x86_64-linux
14   - [ ] aarch64-linux
15   - [ ] x86_64-darwin
16   - [ ] aarch64-darwin
17 - [ ] For non-Linux: Is `sandbox = true` set in `nix.conf`? (See [Nix manual](https://nixos.org/manual/nix/stable/command-ref/conf-file.html))
18 - [ ] Tested, as applicable:
19   - [NixOS test(s)](https://nixos.org/manual/nixos/unstable/index.html#sec-nixos-tests) (look inside [nixos/tests](https://github.com/NixOS/nixpkgs/blob/master/nixos/tests))
20   - and/or [package tests](https://nixos.org/manual/nixpkgs/unstable/#sec-package-tests)
21   - or, for functions and "core" functionality, tests in [lib/tests](https://github.com/NixOS/nixpkgs/blob/master/lib/tests) or [pkgs/test](https://github.com/NixOS/nixpkgs/blob/master/pkgs/test)
22   - made sure NixOS tests are [linked](https://nixos.org/manual/nixpkgs/unstable/#ssec-nixos-tests-linking) to the relevant packages
23 - [ ] Tested compilation of all packages that depend on this change using `nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"`. Note: all changes have to be committed, also see [nixpkgs-review usage](https://github.com/Mic92/nixpkgs-review#usage)
24 - [ ] Tested basic functionality of all binary files (usually in `./result/bin/`)
25 - [22.05 Release Notes (or backporting 21.11 Release notes)](https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md#generating-2205-release-notes)
26   - [ ] (Package updates) Added a release notes entry if the change is major or breaking
27   - [ ] (Module updates) Added a release notes entry if the change is significant
28   - [ ] (Module addition) Added a release notes entry if adding a new NixOS module
29   - [ ] (Release notes changes) Ran `nixos/doc/manual/md-to-db.sh` to update generated release notes
30 - [ ] Fits [CONTRIBUTING.md](https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md).
32 <!--
33 To help with the large amounts of pull requests, we would appreciate your
34 reviews of other pull requests, especially simple package updates. Just leave a
35 comment describing what you have tested in the relevant package/service.
36 Reviewing helps to reduce the average time-to-merge for everyone.
37 Thanks a lot if you do!
39 List of open PRs: https://github.com/NixOS/nixpkgs/pulls
40 Reviewing guidelines: https://nixos.org/manual/nixpkgs/unstable/#chap-reviewing-contributions
41 -->