<div dir="ltr">Just wanted to thank everyone that’s voted or replied already. Some great discussions on this proposal so far.<div><br></div><div>There are four more calendar days left to vote. So if you haven’t voted yet, please do so in the couple days.</div><div><br></div><div>Thank you.</div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><br></div><div>David<br></div></div></div></div></div></div></div></div>
<br><div class="gmail_quote">On Tue, May 30, 2017 at 3:07 PM, Eric Helms <span dir="ltr"><<a href="mailto:ehelms@redhat.com" target="_blank">ehelms@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">As an outside contributor, I'd +1 switching to cherry picking to avoid having this conversation everytime a contributor wants to clone the repo, make a change, push the change [1]. I think you increase the likelihood of first time and repeat contributions if users do not have to be aware of the release process and can simply clone the repository, get the default branch which most developers these days perceive as the head of development and where changes should be made.<div><br></div><div>As someone who has managed project release that heavily use cherry picking, I've found this process to be painless especially after a bit of tooling that uses Redmine to tell me what set of issues are required, what are their changesets and what git hashes are associated with them. Two commands and cherry picking done.<br><div><br></div><div><br></div><div>[1] <a href="https://github.com/pulp/pulp/pull/3037" target="_blank">https://github.com/pulp/<wbr>pulp/pull/3037</a></div></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 30, 2017 at 1:30 PM, Bihan Zhang <span dir="ltr"><<a href="mailto:bizhang@redhat.com" target="_blank">bizhang@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">+1 I think it's worth giving this new process a try.<div>An additional benefit that I have not seen listed is that github push protection can be turned on to all the branches. Which would have someone else sanity check changes before merging.<div><div class="gmail_extra"><br></div><div class="gmail_extra">It also means each release only generates one additional commit instead of the 6+ commits per release generated now.</div><div><div class="m_-6785121958265252924h5"><div class="gmail_extra"><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 30, 2017 at 10:11 AM, Jeff Ortel <span dir="ltr"><<a href="mailto:jortel@redhat.com" target="_blank">jortel@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+1<br>
<div class="m_-6785121958265252924m_7908400682060026998m_4173238035963482565HOEnZb"><div class="m_-6785121958265252924m_7908400682060026998m_4173238035963482565h5"><br>
On 05/24/2017 03:00 PM, David Davis wrote:<br>
> I’d like to kick off the voting on PUP-3 which is the proposal to change our git workflow by using<br>
> cherry-picks instead of merging changes forward. The proposal can be viewed here:<br>
><br>
> <a href="https://github.com/daviddavis/pups/blob/pup3/pup-0003.md" rel="noreferrer" target="_blank">https://github.com/daviddavis/<wbr>pups/blob/pup3/pup-0003.md</a><br>
><br>
> Feel free to submit any comments/nitpicks/etc on the PR[0].<br>
><br>
> PUP-1 outlines our voting system:<br>
><br>
> <a href="https://github.com/pulp/pups/blob/master/pup-0001.md" rel="noreferrer" target="_blank">https://github.com/pulp/pups/b<wbr>lob/master/pup-0001.md</a><br>
><br>
> But to sum it up:<br>
><br>
> +1: "Will benefit the project and should definitely be adopted."<br>
> +0: "Might benefit the project and is acceptable."<br>
> -0: "Might not be the right choice but is acceptable."<br>
> -1: "Not the right choice and should definitely not be adopted."<br>
><br>
> I’ll set the initial deadline for the voting to be June 5th 9pm UTC.<br>
><br>
> [0] <a href="https://github.com/pulp/pups/pull/3" rel="noreferrer" target="_blank">https://github.com/pulp/pups/p<wbr>ull/3</a><br>
><br>
> David<br>
><br>
><br>
</div></div><div class="m_-6785121958265252924m_7908400682060026998m_4173238035963482565HOEnZb"><div class="m_-6785121958265252924m_7908400682060026998m_4173238035963482565h5">> ______________________________<wbr>_________________<br>
> Pulp-dev mailing list<br>
> <a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
> <a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman<wbr>/listinfo/pulp-dev</a><br>
><br>
<br>
</div></div><br>______________________________<wbr>_________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman<wbr>/listinfo/pulp-dev</a><br>
<br></blockquote></div><br></div></div></div></div></div></div>
<br>______________________________<wbr>_________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman<wbr>/listinfo/pulp-dev</a><br>
<br></blockquote></div><br></div>
</div></div><br>______________________________<wbr>_________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/<wbr>mailman/listinfo/pulp-dev</a><br>
<br></blockquote></div><br></div>