PRs already reviewed

Yeah, though reference to PRs ready for review _ - #30 by lilyball it’s not even May anymore :smile:

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.

Merge - plata-theme: 0.8.7 -> 0.8.8 by tadfisher · Pull Request #65579 · NixOS/nixpkgs · GitHub

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 :smiley:

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: https://github.com/NixOS/nixpkgs/pull/63013

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

Merge: spotifyd: update cargoSha256 hash by BenSchZA · Pull Request #66233 · NixOS/nixpkgs · GitHub

1 Like

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)

1 Like

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.

https://github.com/NixOS/nixpkgs/pull/67227

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 :slight_smile:

1 Like

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

~~https://github.com/NixOS/nixpkgs/pull/69576~~

I have some:

https://github.com/NixOS/nixpkgs/pull/70573

Updates qtcreator. The change used to be a single line version update. However, the last time, github:@deadloko added a piece of code to add some functionality. He is the author of this PR as well and to me his changes in this commit seem consistent with the changes in previous commit.

Besides, the maintainer has reviewed and approved.

Update of cups for staging: ~~https://github.com/NixOS/nixpkgs/pull/70969~~
Tested and reviewed.

1 Like

merge: ~~https://github.com/NixOS/nixpkgs/pull/72398~~

merge: ~~https://github.com/NixOS/nixpkgs/pull/71507~~

merge: ~~https://github.com/NixOS/nixpkgs/pull/72187~~

merge: smplayer: 19.5.0 -> 19.10.0 by r-ryantm · Pull Request #72660 · NixOS/nixpkgs · GitHub

merge: du-dust: 0.2.3 -> 0.4.1.2 by marsam · Pull Request #72688 · NixOS/nixpkgs · GitHub

1 Like

merge: gnash: fix compilation with libgif 5.2 by marius851000 · Pull Request #73040 · NixOS/nixpkgs · GitHub

merge: ~~https://github.com/NixOS/nixpkgs/pull/73057~~