hickory icon indicating copy to clipboard operation
hickory copied to clipboard

Close pull requests that won't be merged

Open ricardojmendez opened this issue 8 years ago • 3 comments

As a suggestion, having pull requests open for years that have neither been merged nor closed gives the impression that the project might be abandoned, and could discourage future pull requests (fwiw, it almost stopped me).

I'd recommend reviewing the older ones and deciding if you want to keep them open, particularly if they haven't had any activity in a while.

ricardojmendez avatar Oct 31 '16 06:10 ricardojmendez

Yeah, Maybe you're right. My feeling has always been that they're there if the original requester wants to pick it back up, or if someone else wants to come along and do the work.

davidsantiago avatar Oct 31 '16 18:10 davidsantiago

or if someone else wants to come along and do the work.

I'd say that's a reasonable expectation if it's an issue, but for a PR, either it's merged or it isn't. If someone's considering submitting a new one, and they see PRs open from 2014 in 2016, it gives the impression the project's not following up.

How about adding a note to see if the submitter wants to bring them up to speed, and otherwise just close them after X time of radio silence? Might not even be clear to the submitter what the requirements for acceptance are.

ricardojmendez avatar Oct 31 '16 19:10 ricardojmendez

As a contributor, I second that more feedback would be helpful. It seemed that my PR #33 was on track to be merged, but it has been in limbo for a while even if the PR would remove a bottleneck for using hickory dependents on Node.

njordhov avatar Dec 13 '16 20:12 njordhov

100% agree with the sentiment, will work on it

port19x avatar Apr 11 '23 08:04 port19x

Not sure how to proceed with #35, but it's plausible enough to leave open

port19x avatar Apr 11 '23 09:04 port19x