Reviews older than 90 days

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello folks,

I put this up for discussion at the last community meeting, but I was unable to
attend the meeting. We have a lot of reviews that are quite old[1]. They have,
at some point, become unmergable or the person who proposed them lost interest.
This is on us. We've let reviews slip through the cracks. There are also the
odd cases when a second patch has been merged which fixes the actual issue, but
the old one isn't abandoned.

There's no way we can work our way through the entire list. I suggest that we
abandon all patches with no updates in the last 90 days. If it hits on a patch
that someone is particularly passionate to get merged in, they're welcome to
re-open it.

A. Current master dashboard: https://review.gluster.org/#/projects/glusterfs,dashboards/dashboard:master-dashboard
B. Master dashboard (older than 90 days): http://bit.ly/2m8FX6m
C. Master dashboard (only patches newer than 90 days): http://bit.ly/2mB0xsl

If we abandon older patches, our master dashboard will look like (C) above.
This means that there's already patches that are quite a few reviews that need
some hand holding. Is it possible for us to catch up to the ones that are
currently pending?

[1]: http://bit.ly/2nfBq1N

--
nigelb

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
http://lists.gluster.org/mailman/listinfo/gluster-devel

[Index of Archives]     [Gluster Users]     [Ceph Users]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux