Here are 2 derivations that got reviewed and should be ready to merge :
https://github.com/NixOS/nixpkgs/pull/88012
https://github.com/NixOS/nixpkgs/pull/88821
The first one is mine, the second is just one I would like to see merged.
Here’s an easy version bump and minor tweak for a package and NixOS module I maintain:
Add many extra spaCy models:
https://github.com/NixOS/nixpkgs/pull/91217
https://github.com/NixOS/nixpkgs/pull/90713
[Merged] Package chewing-editor, the user phrase editor of Chewing Input Method.
A minor but important piece of application for Chewing users, especially for there is a problem in the current version of libchewing that adds incorrect user phrases to the database. (chewing/libchewing issue #232)
https://github.com/NixOS/nixpkgs/pull/91270
Three of mine here that have been languishing a little…
Bunch of speed and space fixes for dockerTools.buildLayeredImage: dockerTools: add streaming image support, improve speed and reduce IO by purcell · Pull Request #91084 · NixOS/nixpkgs · GitHub
Fairly straightforward version bump and fix for the fast-downward planner: fast-downward: 2019-05-13 -> 19.12, build on darwin, fix runtime issues by purcell · Pull Request #90281 · NixOS/nixpkgs · GitHub
Trivially prevent netcdf from depending on gcc at runtime: https://github.com/NixOS/nixpkgs/pull/90116
Another controversial one,but this will allow to migrate many options for managing the shell in nixpkgs
I have followed all directions and suggestions on github discussion. I think the PR is ready to merge.
This simple PR has already my review but another pair of eyes would be appreciated.
I’d benefit of some review of people who use the VM building facilities in NixOS:
It would be great if someone could help me get this pr merged
https://github.com/NixOS/nixpkgs/pull/89322
Already peer reviewed, needs final review + merger. Thanks.
Alternative to this Thread (experimental)
If your PR is ready for review, consider testing the new bot Marvin. You can opt-in your PR and indicate that you’re looking for a reviewer by posting a comment with the commands
/marvin opt-in
/status needs_reviewer
The bot will then set the appropriate status label. It will also attempt to assign you a reviewer, though review capacity is limited. If it does not assign you a reviewer right away, please be a bit patient. The bot will assign reviewers to older PRs first, your PR will get its turn eventually. Once it has been reviewed, it will be brought to the attention of someone who can merge it.
If your PR is managed by marvin and gets stuck waiting for a review, that is considered a bug. You can only opt-in your own PRs.
Marvin is still experimental.
5 Likes
hello, would be nice If someone could test the changes in make-disk-image: add hybrid and dynamic sized images by Lassulus · Pull Request #89331 · NixOS/nixpkgs · GitHub
cheers