[Github-comments] [geany/geany] build.c: Remove g_ptr_array_foreach with gpointer user_data & update HACKING (#2270)

[Nick Treleaven]

> DO NOT MERGE PRs while there are still discussions

As Matthew said, the discussion was about foreach_, which the merged code no longer used.

> neither of us nominated reviewers had approved.

It seemed Matthew agreed with the change because he suggested wording for the HACKING file. He hadn't commented on the build.c change, but as he appeared to support the principle in the HACKING file, the build.c change was enforcing the principle.

BTW, I'm not sure what you mean about 'nominated reviewers'?

> I am tempted to revert on principle

You had told me a few months ago to leave pull requests open for at least a week, including a full weekend, before merging (I would still wait longer for any change likely to need discussion). I hadn't seen any new criteria for merging since then. Reverting should only be done if there's a commit that causes more harm than good, and the author is not able to fix it speedily.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2270#issuecomment-526667531
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20190830/9263949f/attachment-0001.html>