PRs already reviewed

I’m posting here my own PR, because I got 2 positive reviews, but I think it got lost:
merge: https://github.com/NixOS/nixpkgs/pull/63942

1 Like

Close: https://github.com/NixOS/nixpkgs/issues/39388
Close: https://github.com/NixOS/nixpkgs/pull/52632

Reason: These have been obsoleted and resolved by #60250.

Close: https://github.com/NixOS/nixpkgs/issues/61672

Close: https://github.com/NixOS/nixpkgs/pull/64510
Merge: https://github.com/NixOS/nixpkgs/pull/64482

Merge: https://github.com/NixOS/nixpkgs/pull/64614

Merge: https://github.com/NixOS/nixpkgs/pull/61438

looks good, already been reviewed and the comments addressed.

Two approvals: https://github.com/NixOS/nixpkgs/pull/63673

Merge: https://github.com/NixOS/nixpkgs/pull/64186
Merge: https://github.com/NixOS/nixpkgs/pull/65094

Merge: https://github.com/NixOS/nixpkgs/pull/64932

Merge: https://github.com/NixOS/nixpkgs/pull/65234

It would be nice if someone how had some awareness about mpd to review the following PR

review: https://github.com/NixOS/nixpkgs/pull/57608

And a slightly less invasive version https://github.com/NixOS/nixpkgs/pull/62771

1 Like

That would probably reach a better target audience in PRs ready for review [May 2019] :slight_smile:

Yeah, though reference to PRs ready for review [May 2019] 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 - https://github.com/NixOS/nixpkgs/pull/65579

I need some help/review for updating vagrant : https://github.com/NixOS/nixpkgs/pull/64302

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?): https://github.com/NixOS/nixpkgs/pull/39060
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: https://github.com/NixOS/nixpkgs/pull/65890
Close old alternative: https://github.com/NixOS/nixpkgs/pull/44494