Recent merge requests in need of attention

[sthursfield (Sam Thursfield)]

Hello,
I haven’t posted this list to discourse for a few weeks. Remember you can always see the list at: https://gnome-metrics.afuera.me.uk/. We seem to have fewer MRs going more than a week without comments, which is great to see!

gupnp-igd - add JavaScript example

Opened: Feb 14
Author: sonny
URL: https://gitlab.gnome.org/GNOME/gupnp-igd/-/merge_requests/5

GNOME Tweaks - CssProvider.load_from_data takes bytes, not string

Opened: Apr 03 2020 (updated Feb 13)
Author: GrbavaCigla
URL: https://gitlab.gnome.org/GNOME/gnome-tweaks/-/merge_requests/55

gtk - Improve FileChooser a11y

Opened: Oct 19 (updated Feb 12)
Author: ydirson
URL: https://gitlab.gnome.org/GNOME/gtk/-/merge_requests/2721

gtk-doc - Add optional dark theme

Opened: Feb 10
Author: vitaly-zdanevich
URL: https://gitlab.gnome.org/GNOME/gtk-doc/-/merge_requests/64

nautilus-python - nautilus-python-object.c: Remove the extra reference on the PyObject file wrappers when adding them to a python list.

Opened: Feb 03
Author: miketwebster
URL: https://gitlab.gnome.org/GNOME/nautilus-python/-/merge_requests/8

caribou - Fix subkey popmenu not showing after being dismissed

Opened: Jan 28
Author: ItzSwirlz
URL: https://gitlab.gnome.org/GNOME/caribou/-/merge_requests/5

gnome-session - Helpful diagnosis

Opened: Jan 20
Author: HinTak
URL: https://gitlab.gnome.org/GNOME/gnome-session/-/merge_requests/59

Settings - Add a propper test sound before resorting to a white noise file

Opened: Dec 24 (updated Jan 05)
Author: tsbatista
URL: https://gitlab.gnome.org/GNOME/gnome-control-center/-/merge_requests/924

planner - Native Language Support

Opened: Oct 29 (updated Jan 03)
Author: ahmed.baizid.0
URL: https://gitlab.gnome.org/GNOME/planner/-/merge_requests/15

gnome-photos - Delete Pending Photos on Close

Opened: Jan 02
Author: AbanoubGhadban
URL: https://gitlab.gnome.org/GNOME/gnome-photos/-/merge_requests/156

[ptomato (Philip Chimento)]

@sthursfield

We seem to have fewer MRs going more than a week without comments, which is great to see!

I wonder, with some embarrassment, whether this is because in one of the previous rounds I commented on a larger than usual number of merge requests thanking the contributor and saying that it looked fine to me but I couldn’t merge it because I’m not a maintainer … now I’m wondering if that only serves to take them off of this list without helping!

A more specific question: Improve FileChooser a11y (!2721) · Merge requests · GNOME / gtk · GitLab seems like it’s quite specialized and will require review from someone with expertise in accessibility. It’s also been waiting for several months How should we best go about finding someone to review it?

[ebassi (Emmanuele Bassi)]

@ptomato

How should we best go about finding someone to review it?

I’ve been looking at it, but it’s a complicated beast.

[sthursfield (Sam Thursfield)]

@ptomato

now I’m wondering if that only serves to take them off of this list without helping!

My goal with this initiative was to avoid MRs being ignored completely. So I think you have helped a lot here, and thanks for that!

We can be more ambitious and think about goals for having patches actually reviewed within a certain time. Realistically that depends on having more active maintainers, so we need to look at it from a different angle: how do we make people want to maintain GNOME software, and how do we teach them to do it.