I think we should change stale-bot strategy to auto close PR, If 30 days is too short for contributions, is 60 or 90 days make sence?
In addition, I notice that we have some PR pass CI but none review it or let a suggest on it. So could we add a bot auto remind committer if PR pass CI but no one review? Or remind author if CI failed? Does it make sence? Best wish. -- Jiajie ________________________________ From: airflowuser <[email protected]> Sent: Tuesday, April 23, 2019 16:39 To: [email protected] Subject: Re: Proposal: Automatically mark stale PRs in github Since there are many many open PRs in the repo it can be hard for committers to keep track (I think that you are keeping tack by the mailing list which sometimes can easily be missed). It may be easier to tack using the filter of recently updated (see image) I hoped that some day this will be the default order of PRs. That way activity in a PR from the last page would bump it to the front. Sent with ProtonMail Secure Email. ‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐ On Tuesday, April 23, 2019 11:32 AM, Ash Berlin-Taylor <[email protected]> wrote: > As a user/reporter on other opensource projects I would personally see > auto-close after 30 days to be far too aggressive to the point of being > unfriendly to contributions. > > Unless we get markedly better at merging PRs I wouldn't want to see us mark > as stale so quickly. > > -ash > > > On 22 Apr 2019, at 22:07, Jarek Potiuk [email protected] wrote: > > Here is a better search showing all the 103 issues - all of them "updated" > > 17 days ago > > https://github.com/apache/airflow/pulls?page=1&q=is%3Apr+is%3Aopen+updated%3A<2019-04-06+sort%3Aupdated-desc > > On Mon, Apr 22, 2019 at 11:06 PM Jarek Potiuk [email protected] > > wrote: > > > > > I think current stalebot configuration will not help us for quite a while > > > for mysterious reason. > > > I looked at the current PRs and somehow mysteriously vast majority of > > > issues (even issues last-commented in 2017) have been updated 17 days ago. > > > https://drive.google.com/file/d/19GF1fdpYa2Tf25N3XgAEKrdXBwr9mNH9/view?usp=sharing > > > It looks like they were all updated on 6th of April, at 00:13 CEST. > > > There are 103 such issues: > > > https://github.com/apache/airflow/pulls?utf8=✓&q=is%3Apr+is%3Aopen+updated%3A<2019-04-06+. > > > It would be nice to find out why this happened. > > > From stalebot documentation: "Any change to an issues and pull request is > > > considered an update, including comments, changing labels, applying or > > > removing milestones, or pushing commits.". I think none of that happened > > > to > > > most of the 103 issues (i checked a few and could not find any trace of > > > any > > > such changes). But maybe someone can recall something that happened 6th of > > > April around midnight (Saturday). > > > Current configuration of stalebot (.github/stalebot.yaml) says: 45 days > > > (mark as stakle) and further 7 days (closing). So those issues will be > > > marked as stale by the stalebot around May 20th (providing that such > > > update > > > won't happen again). > > > Maybe then we can set it to 20 days + 7 for now to stale most issues up > > > in 3 days and delete them 10 days from now? If the config will be too > > > aggressive we can change it back after the 103 issues are cleaned-up. > > > J. > > > On Thu, Apr 18, 2019 at 7:54 AM airflowuser > > > [email protected] wrote: > > > > > > > It's already on (or at least was on in December 2018). > > > > In any case here is a list of old PRs that are waiting for committers. > > > > [AIRFLOW-1956] Add parameter whether the navbar clock time is UTC > > > > https://github.com/apache/airflow/pull/2906 > > > > Status: ash commented but there are no further instructions. > > > > [AIRFLOW-620] Feature to tail custom number of logs instead of rendering > > > > whole log > > > > https://github.com/apache/airflow/pull/3992 > > > > Status: Pushed changed in Jan 2019 that were not reviewed > > > > AIRFLOW-3149 Support dataproc cluster deletion on ERROR > > > > https://github.com/apache/airflow/pull/4064 > > > > Status: pushed changes today. CI passed. > > > > [AIRFLOW-1424] make the next execution date of DAGs visible > > > > https://github.com/apache/airflow/pull/2460 > > > > Status: not sure. Waiting for ash ? > > > > [AIRFLOW-1488] Add the TriggeredDagRunSensor operator > > > > https://github.com/apache/airflow/pull/4291 > > > > Status: Waiting for code review > > > > ‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐ > > > > On Thursday, April 18, 2019 12:01 AM, Daniel Imberman < > > > > [email protected]> wrote: > > > > > > > > > As part of our effort to reduce the PR backlog I wanted to proposed > > > > > that > > > > > we set the github stale action https://github.com/apps/stale. This > > > > > will > > > > > allow us to temporarily close PRs/tickets that are not actively being > > > > > worked on. > > > > > (note that this will not remove PRs, it will simply mark PRs as stale > > > > > to > > > > > make it easier for committers) > > > > > > -- > > > Jarek Potiuk > > > Polidea https://www.polidea.com/ | Principal Software Engineer > > > M: +48 660 796 129 <+48660796129> > > > E: [email protected] > > > > -- > > Jarek Potiuk > > Polidea https://www.polidea.com/ | Principal Software Engineer > > M: +48 660 796 129 <+48660796129> > > E: [email protected]
