Completely agree re: not wishing to be pedantic and I know this represents such a tiny part of the overall scope of the project (ie it's much more important that we have a full library vs a perfect commit history).<br><br>I'll try and put together a couple of scenarios based on the current history and how they could be rebased to follow your guidelines. I'll shoot these back to the list for everyone to look over. I'll also try cleaning up a couple of PR's in the backlog and pushing them up to my own fork just to walk people through they could be a bit cleaner.<br><br>All of this aside, it's really quite amazing how well this project is coming along. Thanks for all your involvement.<br><br>Cheers,<br>Tom<br><div class="gmail_quote"><div dir="ltr">On Thu, 17 Dec 2015 at 8:58 AM, Tony Parker <<a href="mailto:anthony.parker@apple.com">anthony.parker@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Hi Tom,<div><br></div><div>I’ve actually been thinking about this a lot over the past few days, but haven’t come to any real conclusion yet.</div><div><br></div><div>The way I’ve traditionally worked on Foundation is that each commit has more content, and each commit is (almost) always associated with a bug. This is basically the process I had in mind when I wrote the CONTRIBUTING document.</div><div><br></div><div>However, we’re at such an early stage in this project now that I don’t want to add a lot of overhead to PRs that contain basic functionality. I’m also grateful for all of the work that everyone is putting in, and don’t want to put off potential contributors with an overly pedantic policy. We’re trying as hard as we can to keep up with everything, but some PRs are sitting for some time before we can fully review them, which means they are inevitably out of date and have to be rebased.</div><div><br></div><div>I think that as more projects start to depend upon the library we will have to be much more cautious about how we accept changes, how they are tested, and how they are structured (e.g., for easier reverting).</div><div><br></div><div>I welcome additional thoughts on this topic.</div><div><br></div><div>Thanks,</div><div>- Tony</div><div><br><div><blockquote type="cite"></blockquote></div></div></div><div style="word-wrap:break-word"><div><div><blockquote type="cite"><div>On Dec 17, 2015, at 1:25 AM, Tom Jowett via swift-corelibs-dev <<a href="mailto:swift-corelibs-dev@swift.org" target="_blank">swift-corelibs-dev@swift.org</a>> wrote:</div><br></blockquote></div></div></div><div style="word-wrap:break-word"><div><div><blockquote type="cite"><div><div dir="ltr">Hi everyone,<div><br></div><div>I'm wondering if there's a way I can help to keep the commit structure of the repo a bit tidier? I can see the guideline being provided on CONTRIBUTING.md however the current commit history could be a little easier to follow for the average viewer (eb06d19 and b4f6e2b were duplicate commits, e.g) and it seems the rebase before PR element of that guideline could be better followed.</div><div><br></div><div>Perhaps a link in CONTRIBUTING.md to a walkthrough on how to squash and rebase commits (including adding tests appropriately for bisect) could be helpful? Happy to put one together in the context of this project if you think so. Also interested in any other suggestions you might have for how this could be remedied.</div><div><br></div><div>Cheers,</div><div>Tom</div><div><br></div><div><br></div></div>
</div></blockquote></div></div></div><div style="word-wrap:break-word"><div><div><blockquote type="cite"><div><img src="https://u2002410.ct.sendgrid.net/wf/open?upn=U7z7oSqRMSwdl2rYt9JJ4zIFaeg4R-2FxExg9xndDuCbyT8gaMD7pUisalDayrbiR9Zub4-2Fc-2FICn1cVq0FxE-2Bh-2BLzA-2BurVMDYri0YZ2FiYCbMQXldLa1XgakuWrlD8HuDdVbOwT6gzIoBFbH-2Bql10-2BgnRaeiuf7Swoyhs5j1Gxm-2FI1Qxz0gcFVqTyEE6INLKKCVdSCJPTgiWimCkypp6yswkTdwkrTb1m5fkRizcTrEyw-3D" alt="" width="1" height="1" border="0" style="min-height:1px!important;width:1px!important;border-width:0!important;margin-top:0!important;margin-bottom:0!important;margin-right:0!important;margin-left:0!important;padding-top:0!important;padding-bottom:0!important;padding-right:0!important;padding-left:0!important">
_______________________________________________<br>swift-corelibs-dev mailing list<br><a href="mailto:swift-corelibs-dev@swift.org" target="_blank">swift-corelibs-dev@swift.org</a><br><a href="https://lists.swift.org/mailman/listinfo/swift-corelibs-dev" target="_blank">https://lists.swift.org/mailman/listinfo/swift-corelibs-dev</a><br></div></blockquote></div></div></div></blockquote></div>