Revert "mycli: disable checking to fix build" (#349914)
[NixPkgs.git] / .github / PULL_REQUEST_TEMPLATE.md
blob20e260b40894285c096e72199169bcc33bc2c5c1
2 <!--
3 ^ Please summarise the changes you have done and explain why they are necessary here ^
5 For package updates please link to a changelog or describe changes, this helps your fellow maintainers discover breaking updates.
6 For new packages please briefly describe the package or provide a link to its homepage.
7 -->
9 ## Things done
11 <!-- Please check what applies. Note that these are not hard requirements but merely serve as information for reviewers. -->
13 - Built on platform(s)
14   - [ ] x86_64-linux
15   - [ ] aarch64-linux
16   - [ ] x86_64-darwin
17   - [ ] aarch64-darwin
18 - For non-Linux: Is sandboxing enabled in `nix.conf`? (See [Nix manual](https://nixos.org/manual/nix/stable/command-ref/conf-file.html))
19   - [ ] `sandbox = relaxed`
20   - [ ] `sandbox = true`
21 - [ ] Tested, as applicable:
22   - [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))
23   - and/or [package tests](https://github.com/NixOS/nixpkgs/blob/master/pkgs/README.md#package-tests)
24   - 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)
25   - made sure NixOS tests are [linked](https://github.com/NixOS/nixpkgs/blob/master/pkgs/README.md#linking-nixos-module-tests-to-a-package) to the relevant packages
26 - [ ] 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)
27 - [ ] Tested basic functionality of all binary files (usually in `./result/bin/`)
28 - [24.11 Release Notes](https://github.com/NixOS/nixpkgs/blob/master/nixos/doc/manual/release-notes/rl-2411.section.md) (or backporting [23.11](https://github.com/NixOS/nixpkgs/blob/master/nixos/doc/manual/release-notes/rl-2311.section.md) and [24.05](https://github.com/NixOS/nixpkgs/blob/master/nixos/doc/manual/release-notes/rl-2405.section.md) Release notes)
29   - [ ] (Package updates) Added a release notes entry if the change is major or breaking
30   - [ ] (Module updates) Added a release notes entry if the change is significant
31   - [ ] (Module addition) Added a release notes entry if adding a new NixOS module
32 - [ ] Fits [CONTRIBUTING.md](https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md).
34 <!--
35 To help with the large amounts of pull requests, we would appreciate your
36 reviews of other pull requests, especially simple package updates. Just leave a
37 comment describing what you have tested in the relevant package/service.
38 Reviewing helps to reduce the average time-to-merge for everyone.
39 Thanks a lot if you do!
41 List of open PRs: https://github.com/NixOS/nixpkgs/pulls
42 Reviewing guidelines: https://github.com/NixOS/nixpkgs/blob/master/pkgs/README.md#reviewing-contributions
43 -->
45 ---
47 Add a :+1: [reaction] to [pull requests you find important].
49 [reaction]: https://github.blog/2016-03-10-add-reactions-to-pull-requests-issues-and-comments/
50 [pull requests you find important]: https://github.com/NixOS/nixpkgs/pulls?q=is%3Aopen+sort%3Areactions-%2B1-desc