<pre style='margin:0'>
Joshua Root (jmroot) pushed a commit to branch master
in repository macports-guide.
</pre>
<p><a href="https://github.com/macports/macports-guide/commit/1479256a4363218258f591666b78174c9966092d">https://github.com/macports/macports-guide/commit/1479256a4363218258f591666b78174c9966092d</a></p>
<pre style="white-space: pre; background: #F8F8F8">The following commit(s) were added to refs/heads/master by this push:
<span style='display:block; white-space:pre;color:#404040;'> new 1479256 Update and clarify openmaintainer policy
</span>1479256 is described below
<span style='display:block; white-space:pre;color:#808000;'>commit 1479256a4363218258f591666b78174c9966092d
</span>Author: Joshua Root <jmr@macports.org>
AuthorDate: Sat Dec 29 14:03:16 2018 +1100
<span style='display:block; white-space:pre;color:#404040;'> Update and clarify openmaintainer policy
</span><span style='display:block; white-space:pre;color:#404040;'>
</span><span style='display:block; white-space:pre;color:#404040;'> See https://lists.macports.org/pipermail/macports-dev/2018-October/039536.html
</span>---
guide/xml/project.xml | 17 ++++++++++++++++-
1 file changed, 16 insertions(+), 1 deletion(-)
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/guide/xml/project.xml b/guide/xml/project.xml
</span><span style='display:block; white-space:pre;color:#808080;'>index 0684b90..3ed5e51 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/guide/xml/project.xml
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/guide/xml/project.xml
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -1024,9 +1024,24 @@ To use the current port, you must be in a port's directory.</screen>
</span> <listitem>
<para>If a port's maintainer contains the address
<email>openmaintainer@macports.org</email>, this means that the
<span style='display:block; white-space:pre;background:#ffe0e0;'>- author allows minor updates to the port without contacting them
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ author allows minor updates to the port by other committers without
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ contacting them
</span> first. But permission should still be sought for major
changes.</para>
<span style='display:block; white-space:pre;background:#e0ffe0;'>+ <para>Committers are expected to investigate as thoroughly as
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ necessary to confirm that an update is in fact minor. Some
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ projects have made quite major changes with only a tiny change
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ to the version number. And of course a committer should always
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ verify that a port not only builds but works correctly after a
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ change, before pushing it.</para>
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ <para>Pull requests for maintained ports should not be merged
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ by anyone other than
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ their creator or the port maintainer until the 72-hour timeout
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ period has passed, even if the port is openmaintainer. This is
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ because the change is either from a non-committer, or from a
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ committer who could have just pushed the change directly, and
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ by opening a PR is signalling a desire to have the change
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ reviewed by the maintainer.</para>
</span> </listitem>
<listitem>
</pre><pre style='margin:0'>
</pre>