# PRs ready for review \[December\]

**URL:** https://discourse.nixos.org/t/prs-ready-for-review-december/1711
**Category:** Uncategorized
**Created:** [December 19, 2018, 7:33pm UTC](https://discourse.nixos.org/t/prs-ready-for-review-december/1711 "2018-12-19T19:33:54Z")
**Posts on this page:** 20
**Page:** 1

<div class="post-metadata">

### Author: ![matthewbauer](https://discourse.nixos.org/user_avatar/discourse.nixos.org/matthewbauer/32/31_2.png) [@matthewbauer](https://discourse.nixos.org/u/matthewbauer)
#### Post date: [December 19, 2018, 7:33pm UTC](https://discourse.nixos.org/t/prs-ready-for-review-december/1711/1 "2018-12-19T19:33:54Z")

</div>

Hi all,

As a follow up to [PRs ready for review](https://discourse.nixos.org/t/prs-ready-for-review/1016), I wanted to start a thread for anyone to post PRs they have that are ready to be reviewed. We frequently get days where there are more PRs opened than we have the power to review every one. In those cases, things can get lost, even though the content is perfectly good.

Please reply with a link to a PR (can be yours or someone else’s) and I can take a look at it. I am especially looking for concise, finished PRs that are 7+ days old without any significant attention.

---

<div class="post-metadata">

### Author: ![earvstedt](https://discourse.nixos.org/user_avatar/discourse.nixos.org/earvstedt/32/310_2.png) [@earvstedt](https://discourse.nixos.org/u/earvstedt)
#### Post date: [December 19, 2018, 9:54pm UTC](https://discourse.nixos.org/t/prs-ready-for-review-december/1711/2 "2018-12-19T21:54:29Z")

</div>

Very short, but neglected.  
[https://github.com/NixOS/nixpkgs/pull/47665/](https://github.com/NixOS/nixpkgs/pull/47665/)

---

<div class="post-metadata">

### Author: ![bhipple](https://discourse.nixos.org/user_avatar/discourse.nixos.org/bhipple/32/201_2.png) [@bhipple](https://discourse.nixos.org/u/bhipple)
#### Post date: [December 20, 2018, 1:00am UTC](https://discourse.nixos.org/t/prs-ready-for-review-december/1711/3 "2018-12-20T01:00:23Z")

</div>

If we haven’t done so already, we might want to consider enabling stalebot github integration:

- [https://probot.github.io/apps/stale/](https://probot.github.io/apps/stale/)

Nix has a handful of PRs that have not had any activity or comments for many months, including some that haven’t had a comment for over a year. You can sort by least recently updated to take a look:

- [Pull requests · NixOS/nixpkgs · GitHub](https://github.com/NixOS/nixpkgs/pulls?q=is%3Apr+is%3Aopen+sort%3Aupdated-asc)

In the oldest one, the last several months worth of activity are from various maintainers asking if the PR is still desired/relevant.

The above bot can be enabled with a simple click, and only requires write access to pull requests and issues (cannot modify code or commit), plus a small yaml file for config in the main repo. It gives a gentle prod with a comment after some threshold of idle time, then if there’s still no activity a full week later it closes the PR. Since activity can be anything, including another comment, a CI build, commits, a rebase, etc., this is not a high bar to clear. This seems like it’d sweep up some stuff where the original author has disappeared, without maintainers having to manually evaluate and close them out themselves.

I think it’d probably we worth setting up with a relatively long leash (several months at least), because then our issue/PR numbers would more closely reflect the actual number of unreviewed and/or WIP PRs in flight.

---

<div class="post-metadata">

### Author: ![hedning](https://discourse.nixos.org/user_avatar/discourse.nixos.org/hedning/32/113_2.png) [@hedning](https://discourse.nixos.org/u/hedning)
#### Post date: [December 20, 2018, 12:54pm UTC](https://discourse.nixos.org/t/prs-ready-for-review-december/1711/4 "2018-12-20T12:54:06Z")

</div>

Not my PR, but I think we should try out [stdenv: prune libtool files by default by pbogdan · Pull Request #51767 · NixOS/nixpkgs · GitHub](https://github.com/NixOS/nixpkgs/pull/51767) soon. It turns on pruning of .la files by default, and could prevent quite a lot build problems which would otherwise require manual intervention (eg. `strongswan` being broken on master atm.).

---

<div class="post-metadata">

### Author: ![tomberek](https://discourse.nixos.org/user_avatar/discourse.nixos.org/tomberek/32/283_2.png) [@tomberek](https://discourse.nixos.org/u/tomberek)
#### Post date: [December 20, 2018, 6:36pm UTC](https://discourse.nixos.org/t/prs-ready-for-review-december/1711/5 "2018-12-20T18:36:51Z")

</div>

Some NixOps PRs have been sitting for a while. Mine isn’t the most critical, but a feature add that I’m excited about. Been using it for a month and it is very convenient. Feedback appreciated!  
[https://github.com/NixOS/nixops/pull/1048](https://github.com/NixOS/nixops/pull/1048)

---

<div class="post-metadata">

### Author: ![tokudan](https://discourse.nixos.org/letter_avatar_proxy/v4/letter/t/dc4da7/32.png) [@tokudan](https://discourse.nixos.org/u/tokudan)
#### Post date: [December 26, 2018, 11:53pm UTC](https://discourse.nixos.org/t/prs-ready-for-review-december/1711/6 "2018-12-26T23:53:27Z")

</div>

[https://github.com/NixOS/nixpkgs/pull/52645](https://github.com/NixOS/nixpkgs/pull/52645)  
Fixes a broken firmware on some devices that prevents the wlan adapter from working on 18.09

---

<div class="post-metadata">

### Author: ![Anton-Latukha](https://discourse.nixos.org/user_avatar/discourse.nixos.org/anton-latukha/32/227_2.png) [@Anton-Latukha](https://discourse.nixos.org/u/Anton-Latukha)
#### Post date: [January 2, 2019, 2:55pm UTC](https://discourse.nixos.org/t/prs-ready-for-review-december/1711/7 "2019-01-02T14:55:39Z")

</div>

Major update of videocoding utility. Upstream migrated from LibAV to FFMpeg.  
Sorted package dependencies.  
Cleaned-up dependencies.  
[https://github.com/NixOS/nixpkgs/pull/52453/files](https://github.com/NixOS/nixpkgs/pull/52453/files)

---

<div class="post-metadata">

### Author: ![teto](https://discourse.nixos.org/user_avatar/discourse.nixos.org/teto/32/182_2.png) [@teto](https://discourse.nixos.org/u/teto)
#### Post date: [January 7, 2019, 12:44am UTC](https://discourse.nixos.org/t/prs-ready-for-review-december/1711/8 "2019-01-07T00:44:16Z")

</div>

would be thankful to have some feedback on [[RFC] add ability to merge structured configs by teto · Pull Request #42838 · NixOS/nixpkgs · GitHub](https://github.com/NixOS/nixpkgs/pull/42838) . It allows a nixos module to enable kernel settings (if you use a network emulator that requires namespaces, it will enable namespaces). It also makes inheriting kernel configurations easier.

---

<div class="post-metadata">

### Author: ![ma9e](https://discourse.nixos.org/user_avatar/discourse.nixos.org/ma9e/32/528_2.png) [@ma9e](https://discourse.nixos.org/u/ma9e)
#### Post date: [January 7, 2019, 1:47pm UTC](https://discourse.nixos.org/t/prs-ready-for-review-december/1711/9 "2019-01-07T13:47:15Z")

</div>

These add `fuse-overlayfs` and package configurations for `fuse.conf`:  
[https://github.com/NixOS/nixpkgs/pull/53072](https://github.com/NixOS/nixpkgs/pull/53072)  
[https://github.com/NixOS/nixpkgs/pull/53074](https://github.com/NixOS/nixpkgs/pull/53074)

This adds pluggable authentication module support for Gogs/Gitea:  
[https://github.com/NixOS/nixpkgs/pull/52714](https://github.com/NixOS/nixpkgs/pull/52714)

This adds a `davmail` module, which is a gateway for enterprise Microsoft Exchange webservices:  
[https://github.com/NixOS/nixpkgs/pull/52096](https://github.com/NixOS/nixpkgs/pull/52096)

---

<div class="post-metadata">

### Author: ![eonpatapon](https://discourse.nixos.org/user_avatar/discourse.nixos.org/eonpatapon/32/686_2.png) [@eonpatapon](https://discourse.nixos.org/u/eonpatapon)
#### Post date: [January 7, 2019, 4:50pm UTC](https://discourse.nixos.org/t/prs-ready-for-review-december/1711/10 "2019-01-07T16:50:01Z")

</div>

This adds some option to configure cassandra logging

[https://github.com/NixOS/nixpkgs/pull/51570](https://github.com/NixOS/nixpkgs/pull/51570)

---

<div class="post-metadata">

### Author: ![PsyanticY](https://discourse.nixos.org/user_avatar/discourse.nixos.org/psyanticy/32/777_2.png) [@PsyanticY](https://discourse.nixos.org/u/PsyanticY)
#### Post date: [January 13, 2019, 11:22pm UTC](https://discourse.nixos.org/t/prs-ready-for-review-december/1711/11 "2019-01-13T23:22:05Z")

</div>

[https://github.com/NixOS/nixpkgs/pull/50763](https://github.com/NixOS/nixpkgs/pull/50763)

---

<div class="post-metadata">

### Author: ![ericdallo](https://discourse.nixos.org/user_avatar/discourse.nixos.org/ericdallo/32/1954_2.png) [@ericdallo](https://discourse.nixos.org/u/ericdallo)
#### Post date: [December 21, 2019, 4:57pm UTC](https://discourse.nixos.org/t/prs-ready-for-review-december/1711/12 "2019-12-21T16:57:03Z")

</div>

Easy-peasy  
[https://github.com/NixOS/nixpkgs/pull/75124](https://github.com/NixOS/nixpkgs/pull/75124)

---

<div class="post-metadata">

### Author: ![tokudan](https://discourse.nixos.org/letter_avatar_proxy/v4/letter/t/dc4da7/32.png) [@tokudan](https://discourse.nixos.org/u/tokudan)
#### Post date: [December 23, 2019, 1:53pm UTC](https://discourse.nixos.org/t/prs-ready-for-review-december/1711/13 "2019-12-23T13:53:56Z")

</div>

Documentation update:  
[https://github.com/NixOS/nixpkgs/pull/70887](https://github.com/NixOS/nixpkgs/pull/70887)  
and the matching backport to the release notes of 19.09:  
[https://github.com/NixOS/nixpkgs/pull/71056](https://github.com/NixOS/nixpkgs/pull/71056)

---

<div class="post-metadata">

### Author: ![romildo](https://discourse.nixos.org/user_avatar/discourse.nixos.org/romildo/32/223_2.png) [@romildo](https://discourse.nixos.org/u/romildo)
#### Post date: [August 9, 2023, 12:42am UTC](https://discourse.nixos.org/t/prs-ready-for-review-december/1711/15 "2023-08-09T00:42:38Z")

</div>

[https://github.com/NixOS/nixpkgs/pull/246961](https://github.com/NixOS/nixpkgs/pull/246961)

---

<div class="post-metadata">

### Author: ![AkechiShiro](https://discourse.nixos.org/user_avatar/discourse.nixos.org/akechishiro/32/4926_2.png) [@AkechiShiro](https://discourse.nixos.org/u/AkechiShiro)
#### Post date: [September 12, 2023, 4:18am UTC](https://discourse.nixos.org/t/prs-ready-for-review-december/1711/16 "2023-09-12T04:18:06Z")

</div>

> <https://github.com/NixOS/nixpkgs/pull/254281>
>
> \## Description of changes
> 
> \<!--
> For package updates please link to a changelo…g or describe changes, this helps your fellow maintainers discover breaking updates.
> For new packages please briefly describe the package or provide a link to its homepage.
> \--\>
> Changelogs : https://github.com/MicrosoftDocs/azure-docs-cli/blob/main/docs-ref-conceptual/release-notes-azure-cli.md
> \- Bumped azure-mgmt-monitor to from 5.0.0 to 6.0.2
> \## Things done
> 
> 
> 
> \- Built on platform(s)
> - \[X\] x86\_64-linux
> - \[\] aarch64-linux
> - \[\] x86\_64-darwin
> - \[\] aarch64-darwin
> \- \[\] For non-Linux: Is \`sandbox = true\` set in \`nix.conf\`? (See \[Nix manual\](https://nixos.org/manual/nix/stable/command-ref/conf-file.html))
> \- \[X\] Tested, as applicable:
> - \[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))
> - and/or \[package tests\](https://nixos.org/manual/nixpkgs/unstable/#sec-package-tests)
> - 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)
> - made sure NixOS tests are \[linked\](https://nixos.org/manual/nixpkgs/unstable/#ssec-nixos-tests-linking) to the relevant packages
> \- \[X\] 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)
> \- \[X\] Tested basic functionality of all binary files (usually in \`./result/bin/\`)
> \- \[23.11 Release Notes\](https://github.com/NixOS/nixpkgs/blob/master/nixos/doc/manual/release-notes/rl-2311.section.md) (or backporting \[23.05 Release notes\](https://github.com/NixOS/nixpkgs/blob/master/nixos/doc/manual/release-notes/rl-2305.section.md))
> - \[\] (Package updates) Added a release notes entry if the change is major or breaking
> - \[\] (Module updates) Added a release notes entry if the change is significant
> - \[\] (Module addition) Added a release notes entry if adding a new NixOS module
> \- \[X\] Fits \[CONTRIBUTING.md\](https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md).
> 
> \<!--
> To help with the large amounts of pull requests, we would appreciate your
> reviews of other pull requests, especially simple package updates. Just leave a
> comment describing what you have tested in the relevant package/service.
> Reviewing helps to reduce the average time-to-merge for everyone.
> Thanks a lot if you do!
> 
> List of open PRs: https://github.com/NixOS/nixpkgs/pulls
> Reviewing guidelines: https://nixos.org/manual/nixpkgs/unstable/#chap-reviewing-contributions
> \--\>

> <https://github.com/NixOS/nixpkgs/pull/254279>
>
> \## Description of changes
> 
> Pitivi seems to not be compatible with Adwaita and …requires hicolor-theme-icons at runtime.
> \<!--
> For package updates please link to a changelog or describe changes, this helps your fellow maintainers discover breaking updates.
> For new packages please briefly describe the package or provide a link to its homepage.
> \--\>
> 
> \## Things done
> 
> 
> 
> \- Built on platform(s)
> - \[X\] x86\_64-linux
> - \[\] aarch64-linux
> - \[\] x86\_64-darwin
> - \[\] aarch64-darwin
> \- \[X\] For non-Linux: Is \`sandbox = true\` set in \`nix.conf\`? (See \[Nix manual\](https://nixos.org/manual/nix/stable/command-ref/conf-file.html))
> \- \[\] Tested, as applicable:
> - \[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))
> - and/or \[package tests\](https://nixos.org/manual/nixpkgs/unstable/#sec-package-tests)
> - 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)
> - made sure NixOS tests are \[linked\](https://nixos.org/manual/nixpkgs/unstable/#ssec-nixos-tests-linking) to the relevant packages
> \- \[X\] 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)
> \- \[X\] Tested basic functionality of all binary files (usually in \`./result/bin/\`)
> \- \[23.11 Release Notes\](https://github.com/NixOS/nixpkgs/blob/master/nixos/doc/manual/release-notes/rl-2311.section.md) (or backporting \[23.05 Release notes\](https://github.com/NixOS/nixpkgs/blob/master/nixos/doc/manual/release-notes/rl-2305.section.md))
> - \[\] (Package updates) Added a release notes entry if the change is major or breaking
> - \[\] (Module updates) Added a release notes entry if the change is significant
> - \[\] (Module addition) Added a release notes entry if adding a new NixOS module
> \- \[X\] Fits \[CONTRIBUTING.md\](https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md).
> 
> \<!--
> To help with the large amounts of pull requests, we would appreciate your
> reviews of other pull requests, especially simple package updates. Just leave a
> comment describing what you have tested in the relevant package/service.
> Reviewing helps to reduce the average time-to-merge for everyone.
> Thanks a lot if you do!
> 
> List of open PRs: https://github.com/NixOS/nixpkgs/pulls
> Reviewing guidelines: https://nixos.org/manual/nixpkgs/unstable/#chap-reviewing-contributions
> \--\>

Both have been tested, using `nixpkgs-review pr <pr_num>`

---

<div class="post-metadata">

### Author: ![afontaine](https://discourse.nixos.org/user_avatar/discourse.nixos.org/afontaine/32/4623_2.png) [@afontaine](https://discourse.nixos.org/u/afontaine)
#### Post date: [September 12, 2023, 4:24pm UTC](https://discourse.nixos.org/t/prs-ready-for-review-december/1711/17 "2023-09-12T16:24:37Z")

</div>

> <https://github.com/NixOS/nixpkgs/pull/254314>
>
> Looks like the Go version required was bumped to 1.21, and quite a few
> changes:
> …
> https://github.com/superfly/flyctl/compare/v0.1.84...v0.1.90
> 
> \## Things done
> 
> 
> 
> \- Built on platform(s)
> - \[x\] x86\_64-linux
> - \[\] aarch64-linux
> - \[\] x86\_64-darwin
> - \[\] aarch64-darwin
> \- \[\] For non-Linux: Is \`sandbox = true\` set in \`nix.conf\`? (See \[Nix manual\](https://nixos.org/manual/nix/stable/command-ref/conf-file.html))
> \- \[x\] Tested, as applicable:
> - \[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))
> - and/or \[package tests\](https://nixos.org/manual/nixpkgs/unstable/#sec-package-tests)
> - 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)
> - made sure NixOS tests are \[linked\](https://nixos.org/manual/nixpkgs/unstable/#ssec-nixos-tests-linking) to the relevant packages
> \- \[x\] 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)
> \- \[x\] Tested basic functionality of all binary files (usually in \`./result/bin/\`)
> \- \[23.11 Release Notes\](https://github.com/NixOS/nixpkgs/blob/master/nixos/doc/manual/release-notes/rl-2311.section.md) (or backporting \[23.05 Release notes\](https://github.com/NixOS/nixpkgs/blob/master/nixos/doc/manual/release-notes/rl-2305.section.md))
> - x \] (Package updates) Added a release notes entry if the change is major or breaking
> - \[\] (Module updates) Added a release notes entry if the change is significant
> - \[\] (Module addition) Added a release notes entry if adding a new NixOS module
> \- \[x\] Fits \[CONTRIBUTING.md\](https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md).
> 
> \<!--
> To help with the large amounts of pull requests, we would appreciate your
> reviews of other pull requests, especially simple package updates. Just leave a
> comment describing what you have tested in the relevant package/service.
> Reviewing helps to reduce the average time-to-merge for everyone.
> Thanks a lot if you do!
> 
> List of open PRs: https://github.com/NixOS/nixpkgs/pulls
> Reviewing guidelines: https://nixos.org/manual/nixpkgs/unstable/#chap-reviewing-contributions
> \--\>

Tested, and reviewed by one package maintainer.

---

<div class="post-metadata">

### Author: ![honnip](https://discourse.nixos.org/user_avatar/discourse.nixos.org/honnip/32/11100_2.png) [@honnip](https://discourse.nixos.org/u/honnip)
#### Post date: [September 25, 2023, 9:24am UTC](https://discourse.nixos.org/t/prs-ready-for-review-december/1711/18 "2023-09-25T09:24:40Z")

</div>

[https://github.com/NixOS/nixpkgs/pull/255420](https://github.com/NixOS/nixpkgs/pull/255420)

Tested

---

<div class="post-metadata">

### Author: ![SebTM](https://discourse.nixos.org/user_avatar/discourse.nixos.org/sebtm/32/4661_2.png) [@SebTM](https://discourse.nixos.org/u/SebTM)
#### Post date: [April 10, 2024, 5:33pm UTC](https://discourse.nixos.org/t/prs-ready-for-review-december/1711/20 "2024-04-10T17:33:31Z")

</div>

> <https://github.com/NixOS/nixpkgs/pull/250009>
>
> \## Description of changes
> 
> diff: https://github.com/zsh-users/zsh-syntax-highl…ighting/compare/0.7.1...0.8.0\<!--
> For package updates please link to a changelog or describe changes, this helps your fellow maintainers discover breaking updates.
> For new packages please briefly describe the package or provide a link to its homepage.
> \--\>
> 
> \## Things done
> 
> 
> 
> \- Built on platform(s)
> - \[x\] x86\_64-linux
> - \[\] aarch64-linux
> - \[\] x86\_64-darwin
> - \[\] aarch64-darwin
> \- \[\] For non-Linux: Is \`sandbox = true\` set in \`nix.conf\`? (See \[Nix manual\](https://nixos.org/manual/nix/stable/command-ref/conf-file.html))
> \- \[\] Tested, as applicable:
> - \[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))
> - and/or \[package tests\](https://nixos.org/manual/nixpkgs/unstable/#sec-package-tests)
> - 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)
> - made sure NixOS tests are \[linked\](https://nixos.org/manual/nixpkgs/unstable/#ssec-nixos-tests-linking) to the relevant packages
> \- \[\] 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)
> \- \[x\] Tested basic functionality of all binary files (usually in \`./result/bin/\`)
> \- \[23.11 Release Notes\](https://github.com/NixOS/nixpkgs/blob/master/nixos/doc/manual/release-notes/rl-2311.section.md) (or backporting \[23.05 Release notes\](https://github.com/NixOS/nixpkgs/blob/master/nixos/doc/manual/release-notes/rl-2305.section.md))
> - \[\] (Package updates) Added a release notes entry if the change is major or breaking
> - \[\] (Module updates) Added a release notes entry if the change is significant
> - \[\] (Module addition) Added a release notes entry if adding a new NixOS module
> \- \[x\] Fits \[CONTRIBUTING.md\](https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md).
> 
> \<!--
> To help with the large amounts of pull requests, we would appreciate your
> reviews of other pull requests, especially simple package updates. Just leave a
> comment describing what you have tested in the relevant package/service.
> Reviewing helps to reduce the average time-to-merge for everyone.
> Thanks a lot if you do!
> 
> List of open PRs: https://github.com/NixOS/nixpkgs/pulls
> Reviewing guidelines: https://nixos.org/manual/nixpkgs/unstable/#chap-reviewing-contributions
> \--\>
> 
> \# Motivation
> I'm migrating my arch config to nix, and noticed that I get this error with my zsh config:
> \`zsh-syntax-highlighting: unhandled ZLE widget 'zvm\_readkeys\_handler'\`.
> Minimal zsh config for reproducing:
> \`\`\`nix
> home-manager.sharedModules = \[
> {
> programs.zsh = {
> enable = true;
> syntaxHighlighting.enable = true;
> initExtra = ''
> source ${pkgs.zsh-vi-mode}/share/zsh-vi-mode/zsh-vi-mode.plugins.zsh
> # this binds the \<end\> key to end-of-line 
> # and causes the error message but only if old syntax highlighting is enabled
> zvm\_bindkey viins '^\[\[4~' end-of-line
> '';
> }
> }
> \]
> \`\`\`
> Using a newer version of zsh-syntax-highlighting (at least https://github.com/zsh-users/zsh-syntax-highlighting/commit/637e1c702e07b9a3cb76063d27e2118ed3ac1503) fixes the issue.

open a long time with another approval I feel confident to merge

> <https://github.com/NixOS/nixpkgs/pull/298718>
>
> \## Description of changes
> 
> Fixes an issue which prevents registered apps e.g. …nixpkgs-review from successfully offering autocomplete on zsh. (Use it locally as overlay since more than a week 🙏🏻)
> 
> \## Things done
> 
> 
> 
> \- Built on platform(s)
> - \[x\] x86\_64-linux
> - \[\] aarch64-linux
> - \[\] x86\_64-darwin
> - \[\] aarch64-darwin
> \- For non-Linux: Is sandboxing enabled in \`nix.conf\`? (See \[Nix manual\](https://nixos.org/manual/nix/stable/command-ref/conf-file.html))
> - \[\] \`sandbox = relaxed\`
> - \[\] \`sandbox = true\`
> \- \[\] Tested, as applicable:
> - \[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))
> - and/or \[package tests\](https://nixos.org/manual/nixpkgs/unstable/#sec-package-tests)
> - 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)
> - made sure NixOS tests are \[linked\](https://nixos.org/manual/nixpkgs/unstable/#ssec-nixos-tests-linking) to the relevant packages
> \- \[\] 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)
> \- \[x\] Tested basic functionality of all binary files (usually in \`./result/bin/\`)
> \- \[24.05 Release Notes\](https://github.com/NixOS/nixpkgs/blob/master/nixos/doc/manual/release-notes/rl-2405.section.md) (or backporting \[23.05\](https://github.com/NixOS/nixpkgs/blob/master/nixos/doc/manual/release-notes/rl-2305.section.md) and \[23.11\](https://github.com/NixOS/nixpkgs/blob/master/nixos/doc/manual/release-notes/rl-2311.section.md) Release notes)
> - \[\] (Package updates) Added a release notes entry if the change is major or breaking
> - \[\] (Module updates) Added a release notes entry if the change is significant
> - \[\] (Module addition) Added a release notes entry if adding a new NixOS module
> \- \[x\] Fits \[CONTRIBUTING.md\](https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md).
> 
> \<!--
> To help with the large amounts of pull requests, we would appreciate your
> reviews of other pull requests, especially simple package updates. Just leave a
> comment describing what you have tested in the relevant package/service.
> Reviewing helps to reduce the average time-to-merge for everyone.
> Thanks a lot if you do!
> 
> List of open PRs: https://github.com/NixOS/nixpkgs/pulls
> Reviewing guidelines: https://github.com/NixOS/nixpkgs/blob/master/pkgs/README.md#reviewing-contributions
> \--\>
> 
> \---
> 
> 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

has a big number of rebuilds but no upcoming release/commits on master found so might be worth

---

<div class="post-metadata">

### Author: ![Tommimon](https://discourse.nixos.org/letter_avatar_proxy/v4/letter/t/d78d45/32.png) [@Tommimon](https://discourse.nixos.org/u/Tommimon)
#### Post date: [April 29, 2024, 8:39am UTC](https://discourse.nixos.org/t/prs-ready-for-review-december/1711/21 "2024-04-29T08:39:15Z")

</div>

[https://github.com/NixOS/nixpkgs/pull/306216](https://github.com/NixOS/nixpkgs/pull/306216)  
Completion of the work already reviewed and tested on linux.

---

<div class="post-metadata">

### Author: ![nicomem](https://discourse.nixos.org/user_avatar/discourse.nixos.org/nicomem/32/16830_2.png) [@nicomem](https://discourse.nixos.org/u/nicomem)
#### Post date: [May 2, 2024, 6:20pm UTC](https://discourse.nixos.org/t/prs-ready-for-review-december/1711/22 "2024-05-02T18:20:02Z")

</div>

[authelia: 4.37.5 → 4.38.8](https://github.com/NixOS/nixpkgs/pull/299309)  
 → Working well for me (I use it in my home server)

[nixos/luksroot: Exit if EOF detected in `cryptsetup-askpass`](https://github.com/NixOS/nixpkgs/pull/298592)  
 → A small PR that brings some QoL in SSH LUKS unlocking automation

[Next page](https://discourse.nixos.org/t/prs-ready-for-review-december/1711.md?page=2)