Merge requests in need of attention, 18th December 2020

This list shows some first-contribution merge requests which were opened a week or more ago but have not yet received feedback.

I thought we didn’t have many new additions, but in fact my script broke :slight_smile: Here some fresher merge requests,

alacarte - Resize input fields with launcher dialog.

Opened: Dec 09
Author: @SubOptimal
URL: https://gitlab.gnome.org/GNOME/alacarte/-/merge_requests/5

gnome-mines - Added spaces in description for consistency #60

Opened: Dec 08
Author: @DylanRichards
URL: https://gitlab.gnome.org/GNOME/gnome-mines/-/merge_requests/27

libgtop - Add support for GPU metrics

Opened: Dec 06 (updated Dec 08)
Author: @siavashs
URL: https://gitlab.gnome.org/GNOME/libgtop/-/merge_requests/16

libnotify - notify-send: Support commas in icon filenames

Opened: Dec 07
Author: @thorsten_
URL: https://gitlab.gnome.org/GNOME/libnotify/-/merge_requests/15

quadrapassel - fixed issue#2

Opened: Dec 03 (updated Dec 07)
Author: @aliyarahmani786
URL: https://gitlab.gnome.org/GNOME/quadrapassel/-/merge_requests/26

evince - Windows: compilation failed because PathFindExtension() need shlwapi DLL

Opened: Dec 06
Author: @vtorri
URL: https://gitlab.gnome.org/GNOME/evince/-/merge_requests/292

evince - Windows: open() and al. need io.h

Opened: Dec 06
Author: @vtorri
URL: https://gitlab.gnome.org/GNOME/evince/-/merge_requests/293

evince - Windows: fix compilation, CLOEXEC does not exist

Opened: Dec 06
Author: @vtorri
URL: https://gitlab.gnome.org/GNOME/evince/-/merge_requests/291

accerciser - Added Indonesian translation

Opened: Dec 06
Author: @andika
URL: https://gitlab.gnome.org/GNOME/accerciser/-/merge_requests/11

gnome-calendar - Move month forwards/backwards when clicking a date outside of the current month in Month View.

Opened: Dec 04
Author: @cchillen
URL: https://gitlab.gnome.org/GNOME/gnome-calendar/-/merge_requests/154

1 Like

I do wonder what the etiquette is for merging these merge requests — I sometimes have time to review them, but I don’t want to stomp on the maintainers by merging them. I wonder if the script could mark ones that have had a review but not been merged yet after another week or so? Then we could merge those.

That’s an interesting idea, although I can’t immediately see an easy way to implement it. I feel the same as you that I don’t want to incur the wrath of the incumbent maintainer.

One thing I am planning is making a table of “who maintains what” using the DOAP data in the repos. Perhaps there’s some info we could add to this table that’d make it clearer how actively maintained a project is. It would at least make it easier to see who to check with before merging.

You can probably check when was the last non-translation commit, and if the committer was listed in the DOAP file.

The problem is that even if there’s some people set as maintainers in the DOAP, it doesn’t actually mean the project is maintained. There isn’t even anybody to remove those people from the DOAP since there’s no active maintainer :confused:

That’s exactly the kind of thing that I hope we can improve. If the listed maintainers are inactive, then it’s all the more important that the rest of us encourage new contributors, by reviewing and merging things ourselves.

Sure, I agree it’s important to not let contributions unreviewed and un-noticed :wink:

I wonder if the script could mark ones that have had a review but not been merged yet after another week or so? Then we could merge those.

Thinking about this some more, what if I link to the previous week’s post each time I post a new list? That lets us see everything from the previous week, at least.

This topic was automatically closed 14 days after the last reply. New replies are no longer allowed.