Skip to end of metadata
Go to start of metadata

<p>This IRC Meeting is planned for 18:00 UTC on Wednesday, 29 February 2012.</p>

<ac:macro ac:name="info"><ac:parameter ac:name="title">Meeting this week is cancelled</ac:parameter><ac:rich-text-body>
<p>We've cancelled the meeting this week in order to focus on getting the beta3 release completed. We will resume meetings next week.</p></ac:rich-text-body></ac:macro>

<p>Please feel free to add topics, according to the template on the <ac:link><ri:page ri:content-title="IRC Meetings" /><ac:link-body>parent page</ac:link-body></ac:link>.</p>

<h2>Work in Progress (WIP) pull request system</h2>

<p><ac:link><ri:user ri:username="ocramius" /></ac:link> has started discussion about Work in Progress <a href="http://help.github.com/send-pull-requests/">pull requests</a> (PR) <a href="http://zend-framework-community.634137.n4.nabble.com/ZF2-Pull-Request-usage-WIP-Pull-Requests-td4405109.html">on the mailing list</a>. Currently, PRs are used only to merge changes that have already been reworked on foreign repositories and discussed on the mailing list or IRC, while changes not ready for merge end up with a PR being closed. That is not using the full potential of the Github website. He suggested that PR could be kept open even on earlier stages of development to allow longer discussion threads also on Github.</p>

<h5>Pros:</h5>
<ul>
<li>Increased visibility of WIP on ZF2</li>
<li>More contributions by occasional visitors</li>
<li>More discussion and code review through Github (notifications/comments on diffs)</li>
</ul>

<h5>Cons:</h5>
<ul>
<li>Increased number of open PRs</li>
<li>Possible "hanging" of PRs for longer periods of time</li>
</ul>

<p>Some other very different suggestions popped up, but probably a vote during a meeting will be needed for just this point.</p>

<ac:macro ac:name="unmigrated-inline-wiki-markup"><ac:plain-text-body><![CDATA[

WIP pull requests - Should we discuss this? (Log In to vote.)
Choices Your Vote

Yes

No

]]></ac:plain-text-body></ac:macro>

Labels:
None
Enter labels to add to this page:
Please wait 
Looking for a label? Just start typing.