Merge: pythonPackages.diofant: init at 0.10.0 by suhr · Pull Request #65234 · NixOS/nixpkgs · GitHub
It would be nice if someone how had some awareness about mpd to review the following PR
review: mpd: 0.20.13 -> 0.21.16 by tobim · Pull Request #57608 · NixOS/nixpkgs · GitHub
And a slightly less invasive version mpd: nixos module support for standard users by peterhoeg · Pull Request #62771 · NixOS/nixpkgs · GitHub
Yeah, though reference to PRs ready for review _ - #30 by lilyball it’s not even May anymore
An influx of requests to review PRs here because I guess GitHub isn’t really lending it’s hand to this will be problematic eventually. i.e 30 PRs ready for review threads all about the same thing.
Perhaps we need a topic.
Overall the topic seems to be relatively successful. A big thanks to @matthewbauer who I believe stated the idea some time ago. PRs slip through the cracks on github sometimes and until our tagging better identifies these and/or reviewers make use of tagging/searching features a topic like this works well. +1 for renaming/pinning.
I need some help/review for updating vagrant : vagrant: 2.2.3 -> 2.2.5 by angristan · Pull Request #64302 · NixOS/nixpkgs · GitHub
The Gemfile/Gemfile.lock were removed a while back and now it is unclear how to proceed to update the package since it seems to be the usual ruby/gem thing with bundix. I got something working but nobody wants to look at my PR
Close (I think?): error message honores nested dots in attribute path elements by qknight · Pull Request #39060 · NixOS/nixpkgs · GitHub
There are significant merge issues, and there hasn’t been any response on the PR in a few weeks. I’m not sure if that is sufficient to close the PR in this community, though.
Merge: mdbook: 0.1.8 -> 0.3.1 by tazjin · Pull Request #65890 · NixOS/nixpkgs · GitHub
Close old alternative: mdbook: 0.1.8 -> 0.2.0 by dywedir · Pull Request #44494 · NixOS/nixpkgs · GitHub
Close: #64341 (Obsoleted by #66691)
Close: #64501 (Obsoleted by #66691)
Close: #64483 (Obsoleted by #66608)
Close: #62922 (Obsoleted by #65629)
Merge: #66418 (Minor version update, I was using the generated binary for a week)
Merge: #66344 (New package, reviewed, I was using the generated binary for a week)
- wip/merge (suggested option description change) : nixos/transmission: make user configurable by c0deaddict · Pull Request #67393 · NixOS/nixpkgs · GitHub
- wip (could not use the built program) : mpd: 0.20.13 -> 0.21.16 by tobim · Pull Request #57608 · NixOS/nixpkgs · GitHub
edited: I had made a mistake in the links
I have pr I submitted and have been using without issue for a while now. Basically it allows sound.mediaKeys to work with pulseaudio. Doesn’t seem like anyone else has had time to review it but it works just fine and hasn’t caused me any issues. Should be ready to merge. Hopefully someone can take a look.
Thank you for your PR, but please refer to PRs ready for review _ - #46 by pstch (it says May 2019, but it has become the de-facto long running thread) for review requests
merge: dhall-json: fix build by marsam · Pull Request #67438 · NixOS/nixpkgs · GitHub ([haskell] dhall-json: fix build)
merge: Pass hoogle-local's buildCommand as a file. by TravisWhitaker · Pull Request #67228 · NixOS/nixpkgs · GitHub ([haskell] Pass hoogle-local’s buildCommand as a file)
close: pandoc-placetable: enable inlineMarkdown by unode · Pull Request #67464 · NixOS/nixpkgs · GitHub ([haskell] pandoc-placetable: enable inlineMarkdown)
merge: plan9port: use C compiler from Nix by eraserhd · Pull Request #66404 · NixOS/nixpkgs · GitHub - plan9port: use correct C compiler (right?)
The following PR was approved almost 2 weeks ago but is just waiting on being merged