Github messages for voidlinux
 help / color / mirror / Atom feed
* [ISSUE] [RFC] Give stale PRs 2 weeks to show signs of life
@ 2022-03-21  3:17 0x5c
  2022-03-21  3:25 ` subnut
                   ` (10 more replies)
  0 siblings, 11 replies; 12+ messages in thread
From: 0x5c @ 2022-03-21  3:17 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1322 bytes --]

New issue by 0x5c on void-packages repository

https://github.com/void-linux/void-packages/issues/36253

Description:
### ... then nuke the ones that didn't.

## Rationale
The backlog is enormous, scary, and full of PRs that don't meet quality requirements, are waiting (for months) for requested changes, and/or have generally been abandoned. There's even PRs that no longer have an author (#29665, #30909). 
This idea was mentioned multiple times in the IRC channel by multiple people, with a positive response.

## How
1. Set a cutoff amount of inactivity, ie: 6 months
2. Get the list of PRs matching that inactivity to a file, ie: `gh pr list -L 2000 -S 'updated:<=2021-09-20' > file.tsv`
    The issue number is the first column
4. Using a script and the GH API, do for each PR in the list:
    1. Apply the already-existing `inactive` tag
    2. Send a message explaining that the PR will be closed in two weeks if it doesn't get any activity.
5. Wait two weeks
6. Close marked PRs that didn't get any updates.
    This part could be done automatically if searching for `label:inactive updated:<=[date of script run]T[time of script run]`, although it will miss PRs where the activity was just "I don't use void anymore". I suspect this will not be a big problem considering how dead the backlog is.

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [RFC] Give stale PRs 2 weeks to show signs of life
  2022-03-21  3:17 [ISSUE] [RFC] Give stale PRs 2 weeks to show signs of life 0x5c
@ 2022-03-21  3:25 ` subnut
  2022-03-21  3:31 ` 0x5c
                   ` (9 subsequent siblings)
  10 siblings, 0 replies; 12+ messages in thread
From: subnut @ 2022-03-21  3:25 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 481 bytes --]

New comment by subnut on void-packages repository

https://github.com/void-linux/void-packages/issues/36253#issuecomment-1073447657

Comment:
I think first of all we should remove the obsolete PRs, i.e. Those PRs whose newer versions of the package have been accepted already. Then, we should either take over the PRs opened by **ghost** or close them. Then, we should do what has been mentioned above.

Also, I think the wait time should be increased a bit. Three weeks, maybe?

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [RFC] Give stale PRs 2 weeks to show signs of life
  2022-03-21  3:17 [ISSUE] [RFC] Give stale PRs 2 weeks to show signs of life 0x5c
  2022-03-21  3:25 ` subnut
@ 2022-03-21  3:31 ` 0x5c
  2022-03-22 17:58 ` classabbyamp
                   ` (8 subsequent siblings)
  10 siblings, 0 replies; 12+ messages in thread
From: 0x5c @ 2022-03-21  3:31 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 609 bytes --]

New comment by 0x5c on void-packages repository

https://github.com/void-linux/void-packages/issues/36253#issuecomment-1073450049

Comment:
> I think first of all we should remove the obsolete PRs, i.e. Those PRs whose newer versions of the package have been accepted already.

The obsolete PRs are for the most part closed, as tracked in https://github.com/void-linux/void-packages/issues/36116. Both remaining ones have unmerged patches/fixes. The libsigrok one is has a better fix in https://github.com/void-linux/void-packages/pull/36144, but it is yet to be known if the binfmt one is actually needed.

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [RFC] Give stale PRs 2 weeks to show signs of life
  2022-03-21  3:17 [ISSUE] [RFC] Give stale PRs 2 weeks to show signs of life 0x5c
  2022-03-21  3:25 ` subnut
  2022-03-21  3:31 ` 0x5c
@ 2022-03-22 17:58 ` classabbyamp
  2022-03-22 18:04 ` tibequadorian
                   ` (7 subsequent siblings)
  10 siblings, 0 replies; 12+ messages in thread
From: classabbyamp @ 2022-03-22 17:58 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 322 bytes --]

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/issues/36253#issuecomment-1075446917

Comment:
People can still build upon closed PRs, they don't just disappear once closed. The inactive label can be used on closed PRs to differentiate closed=rejected and closed=stale

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [RFC] Give stale PRs 2 weeks to show signs of life
  2022-03-21  3:17 [ISSUE] [RFC] Give stale PRs 2 weeks to show signs of life 0x5c
                   ` (2 preceding siblings ...)
  2022-03-22 17:58 ` classabbyamp
@ 2022-03-22 18:04 ` tibequadorian
  2022-03-22 19:59 ` 0x5c
                   ` (6 subsequent siblings)
  10 siblings, 0 replies; 12+ messages in thread
From: tibequadorian @ 2022-03-22 18:04 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 340 bytes --]

New comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/issues/36253#issuecomment-1075442525

Comment:
I'm not a fan of closing PRs just because of inactivity because it helps others to build upon these.
But I'm all in for tagging these PRs to be able to filter them out using -label:inactive

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [RFC] Give stale PRs 2 weeks to show signs of life
  2022-03-21  3:17 [ISSUE] [RFC] Give stale PRs 2 weeks to show signs of life 0x5c
                   ` (3 preceding siblings ...)
  2022-03-22 18:04 ` tibequadorian
@ 2022-03-22 19:59 ` 0x5c
  2022-03-22 19:59 ` 0x5c
                   ` (5 subsequent siblings)
  10 siblings, 0 replies; 12+ messages in thread
From: 0x5c @ 2022-03-22 19:59 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 647 bytes --]

New comment by 0x5c on void-packages repository

https://github.com/void-linux/void-packages/issues/36253#issuecomment-1075582712

Comment:
@tibequadorian Closing PR also doesn't block anyone from discussing those PRs, nor does it make the PR's content disappear. It all remains there for anyone else to build upon, just without the immense strain on the backlog. If anything, an open PR can discourage others from making new and updated PRs for the same things. And as Abby said, the label doesn't have to be removed from PRs once they are closed for inactivity, and the whole process involves sending a comment explaining why the PR got closed.

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [RFC] Give stale PRs 2 weeks to show signs of life
  2022-03-21  3:17 [ISSUE] [RFC] Give stale PRs 2 weeks to show signs of life 0x5c
                   ` (4 preceding siblings ...)
  2022-03-22 19:59 ` 0x5c
@ 2022-03-22 19:59 ` 0x5c
  2022-03-22 20:14 ` tibequadorian
                   ` (4 subsequent siblings)
  10 siblings, 0 replies; 12+ messages in thread
From: 0x5c @ 2022-03-22 19:59 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 643 bytes --]

New comment by 0x5c on void-packages repository

https://github.com/void-linux/void-packages/issues/36253#issuecomment-1075582712

Comment:
@tibequadorian Closing PRs doesn't block anyone from discussing those PRs, nor does it make the PR's content disappear. It all remains there for anyone else to build upon, just without the immense strain on the backlog. If anything, an open PR can discourage others from making new and updated PRs for the same things. And as Abby said, the label doesn't have to be removed from PRs once they are closed for inactivity, and the whole process involves sending a comment explaining why the PR got closed.

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [RFC] Give stale PRs 2 weeks to show signs of life
  2022-03-21  3:17 [ISSUE] [RFC] Give stale PRs 2 weeks to show signs of life 0x5c
                   ` (5 preceding siblings ...)
  2022-03-22 19:59 ` 0x5c
@ 2022-03-22 20:14 ` tibequadorian
  2022-03-22 21:38 ` 0x5c
                   ` (3 subsequent siblings)
  10 siblings, 0 replies; 12+ messages in thread
From: tibequadorian @ 2022-03-22 20:14 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 475 bytes --]

New comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/issues/36253#issuecomment-1075594774

Comment:
I know the content doesn't disappear but the list of closed PRs (which is 40x bigger and growing) is usually not considered when searching for a useful PR to build upon. Not least because GitHub filters for open PRs automatically. I'd rather just mark them as inactive and keep them open as long as they are useful to others.

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [RFC] Give stale PRs 2 weeks to show signs of life
  2022-03-21  3:17 [ISSUE] [RFC] Give stale PRs 2 weeks to show signs of life 0x5c
                   ` (6 preceding siblings ...)
  2022-03-22 20:14 ` tibequadorian
@ 2022-03-22 21:38 ` 0x5c
  2022-03-22 23:51 ` 0x5c
                   ` (2 subsequent siblings)
  10 siblings, 0 replies; 12+ messages in thread
From: 0x5c @ 2022-03-22 21:38 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 577 bytes --]

New comment by 0x5c on void-packages repository

https://github.com/void-linux/void-packages/issues/36253#issuecomment-1075668575

Comment:
It seems to me it would be more helpful to direct users to the full search, or accept that most of them (especially the new packages) don't meet the quality requirements (of package and or PR), on top of being outdated, and wouldn't be a good base for new PRs. Keep in mind it's not "the old PRs" but inactive PRs which this target; the old but still active PRs generally meet quality requirements and are kept updated by their authors.

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [RFC] Give stale PRs 2 weeks to show signs of life
  2022-03-21  3:17 [ISSUE] [RFC] Give stale PRs 2 weeks to show signs of life 0x5c
                   ` (7 preceding siblings ...)
  2022-03-22 21:38 ` 0x5c
@ 2022-03-22 23:51 ` 0x5c
  2022-04-11  1:39 ` [ISSUE] [CLOSED] " 0x5c
  2022-04-11  1:39 ` 0x5c
  10 siblings, 0 replies; 12+ messages in thread
From: 0x5c @ 2022-03-22 23:51 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 264 bytes --]

New comment by 0x5c on void-packages repository

https://github.com/void-linux/void-packages/issues/36253#issuecomment-1075762279

Comment:
Yet another problem with those PRs: the Github action logs are long gone, meaning we can't even know what the failures were

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [RFC] Give stale PRs 2 weeks to show signs of life
  2022-03-21  3:17 [ISSUE] [RFC] Give stale PRs 2 weeks to show signs of life 0x5c
                   ` (9 preceding siblings ...)
  2022-04-11  1:39 ` [ISSUE] [CLOSED] " 0x5c
@ 2022-04-11  1:39 ` 0x5c
  10 siblings, 0 replies; 12+ messages in thread
From: 0x5c @ 2022-04-11  1:39 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 276 bytes --]

New comment by 0x5c on void-packages repository

https://github.com/void-linux/void-packages/issues/36253#issuecomment-1094457482

Comment:
https://github.com/void-linux/void-packages/pull/36462 has been merged (d6fc5518d55fdc81d75f6ddb72f06e25eeb00196), taking care of this.

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [ISSUE] [CLOSED] [RFC] Give stale PRs 2 weeks to show signs of life
  2022-03-21  3:17 [ISSUE] [RFC] Give stale PRs 2 weeks to show signs of life 0x5c
                   ` (8 preceding siblings ...)
  2022-03-22 23:51 ` 0x5c
@ 2022-04-11  1:39 ` 0x5c
  2022-04-11  1:39 ` 0x5c
  10 siblings, 0 replies; 12+ messages in thread
From: 0x5c @ 2022-04-11  1:39 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1325 bytes --]

Closed issue by 0x5c on void-packages repository

https://github.com/void-linux/void-packages/issues/36253

Description:
### ... then nuke the ones that didn't.

## Rationale
The backlog is enormous, scary, and full of PRs that don't meet quality requirements, are waiting (for months) for requested changes, and/or have generally been abandoned. There's even PRs that no longer have an author (#29665, #30909). 
This idea was mentioned multiple times in the IRC channel by multiple people, with a positive response.

## How
1. Set a cutoff amount of inactivity, ie: 6 months
2. Get the list of PRs matching that inactivity to a file, ie: `gh pr list -L 2000 -S 'updated:<=2021-09-20' > file.tsv`
    The issue number is the first column
4. Using a script and the GH API, do for each PR in the list:
    1. Apply the already-existing `inactive` tag
    2. Send a message explaining that the PR will be closed in two weeks if it doesn't get any activity.
5. Wait two weeks
6. Close marked PRs that didn't get any updates.
    This part could be done automatically if searching for `label:inactive updated:<=[date of script run]T[time of script run]`, although it will miss PRs where the activity was just "I don't use void anymore". I suspect this will not be a big problem considering how dead the backlog is.

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2022-04-11  1:39 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-21  3:17 [ISSUE] [RFC] Give stale PRs 2 weeks to show signs of life 0x5c
2022-03-21  3:25 ` subnut
2022-03-21  3:31 ` 0x5c
2022-03-22 17:58 ` classabbyamp
2022-03-22 18:04 ` tibequadorian
2022-03-22 19:59 ` 0x5c
2022-03-22 19:59 ` 0x5c
2022-03-22 20:14 ` tibequadorian
2022-03-22 21:38 ` 0x5c
2022-03-22 23:51 ` 0x5c
2022-04-11  1:39 ` [ISSUE] [CLOSED] " 0x5c
2022-04-11  1:39 ` 0x5c

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).