<div>Hi all, my thoughts:<br></div><div><br></div><div>What Tom proposes is essentially to use branches instead of tags for code releases. This is in my view a misuse of the system, and at odds with the recommendations from the GitHub people themselves [0]. Marking releases with branches creates three sets of issues:<br></div><div><br></div><div>1. The number of branches increases fast, making the GitHub interface harder to use and complicate branch management in the development environment.<br></div><div><br></div><div>2. Releases become attached to a perishable asset: it is very easy to delete a branch by mistake. In contrast there is no way to delete a tag in the GitHub interface - they were thought to be permanent.<br></div><div><br></div><div>3. A larger number of branches increases the likelihood of pull requests committed against the wrong branch.<br></div><div><br></div><div>For the particular issue at hand I believe the solution is to restrict the branch(es) against which pull requests can be issued. I do not know yet if or how is this possible, I opened a thread about it at StackOverflow [1]. Another thing that can be done right away is to protect the master branch [2], which forces additional checks before pull requests can be merged.<br></div><div><br></div><div>Cheers.<br></div><div><br></div><div>[0] <a href="https://help.github.com/articles/creating-releases/">https://help.github.com/articles/creating-releases/</a><br></div><div>[1] <a href="https://stackoverflow.com/questions/46054696/restricting-branches-to-which-pull-requests-can-be-issued">https://stackoverflow.com/questions/46054696/restricting-branches-to-which-pull-requests-can-be-issued</a><br></div><div class="protonmail_signature_block "><div class="protonmail_signature_block-user "><div>[2] <a href="https://help.github.com/articles/defining-the-mergeability-of-pull-requests/">https://help.github.com/articles/defining-the-mergeability-of-pull-requests/</a><br></div><div><br></div><div>-- <br></div><div>Luís Moreira de Sousa<br></div><div>Im Grund 6<br></div><div>CH-8600 Dübendorf<br></div><div>Switzerland<br></div><div><br></div><div>Phone: +41 (0)79 812 62 65<br></div><div>Email: <a href="mailto:luis.de.sousa@protonmail.ch">luis.de.sousa@protonmail.ch</a><br></div><div>URL: <a href="https://sites.google.com/site/luismoreiradesousa">https://sites.google.com/site/luismoreiradesousa</a><br></div></div><div class="protonmail_signature_block-proton protonmail_signature_block-empty"><br></div></div><div><br></div><blockquote class="protonmail_quote" type="cite"><div>-------- Original Message --------<br></div><div>Subject: Re: [Pywps-dev] git branches confusion (mess)<br></div><div>Local Time: September 2, 2017 7:11 PM<br></div><div>UTC Time: September 2, 2017 5:11 PM<br></div><div>From: tom.kralidis@gmail.com<br></div><div>To: Jachym Cepicky <jachym.cepicky@gmail.com><br></div><div>Luí­s Moreira de Sousa <luis.de.sousa@protonmail.ch>, jorge.dejesus <jorge.dejesus@geocat.net>, pywps-dev@lists.osgeo.org <pywps-dev@lists.osgeo.org><br></div><div><br></div><div>My proposal on Gitter was to keep master as the true dev branch and releasing stable branches on major releases. That seemed like overhead at the time and we agreed to go with a git-flow type model. Sometimes the 'master by default' thinking has PRs being issued to master by mistake.<br></div><div><br></div><div>What do folks think about:<br></div><div><br></div><div>- eliminating develop branch<br></div><div>- working off master as latest<br></div><div>- creating stable branches off major releases<br></div><div><br></div><div>Example for a major release:<br></div><div>- create x.y.z tag<br></div><div>- create x.y stable branch<br></div><div>- bug fixes are applied against master and x.y<br></div><div>- bug fix releases are made against x.y branch<br></div><div>- new features are applied against master<br></div><div><br></div><div>Thoughts?<br></div><div><br></div><div><div>..Tom<br></div><div><br></div><div>Sent from my iPhone<br></div></div><div><div><br></div><div>On Sep 1, 2017, at 15:38, Jachym Cepicky <<a href="mailto:jachym.cepicky@gmail.com">jachym.cepicky@gmail.com</a>> wrote:<br></div></div><blockquote type="cite"><div><div dir="ltr"><div>Hi,<br></div><div><br></div><div>I've just now put PyWPS `master` back to where it belongs 7cab386<br></div><div><br></div><div>Before I did , I merged the code which was added to master to `develop` branch - no code should be lost.<br></div><div><br></div><div>From now on, we have to carefully read, where the pull requests are supposed to be merged and not accept those pull requests, which are done against master.<br></div><div><br></div><div>Related to this:<br></div><div><br></div><div>I would suggest, we open our development policy one more time, seems, people are confused about the fact, that master is stable and not development branch. @TomKralidis had some alternative proposal on Gitter, maybe it could be shared in the mailing list, I would like to have PSC decided about this.<br></div><div><br></div><div>J<br></div></div><div><br></div><div class="gmail_quote"><div dir="ltr">pá 1. 9. 2017 v 15:43 odesílatel Luí­s Moreira de Sousa <<a href="mailto:luis.de.sousa@protonmail.ch">luis.de.sousa@protonmail.ch</a>> napsal:<br></div><blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex" class="gmail_quote"><div>Hi Jachym, that sounds good. But which commits exactly were pushed to the master branch? Depending on that we could have a minor odd release as Jorge suggests.<br></div><div><br></div><div>Cheers.<br></div><div class="m_-1635495664983396338protonmail_signature_block"><div class="m_-1635495664983396338protonmail_signature_block-user"><div><br></div><div>-- <br></div><div>Luís Moreira de Sousa<br></div><div>Im Grund 6<br></div><div>CH-8600 Dübendorf<br></div><div>Switzerland<br></div><div><br></div><div>Phone: <a value="+41798126265" href="tel:+41%2079%20812%2062%2065">+41 (0)79 812 62 65</a><br></div><div>Email: <a href="mailto:luis.de.sousa@protonmail.ch">luis.de.sousa@protonmail.ch</a><br></div><div>URL: <a href="https://sites.google.com/site/luismoreiradesousa">https://sites.google.com/site/luismoreiradesousa</a><br></div></div><div class="m_-1635495664983396338protonmail_signature_block-proton m_-1635495664983396338protonmail_signature_block-empty"><br></div></div><div><br></div><blockquote type="cite" class="m_-1635495664983396338protonmail_quote"><div>-------- Original Message --------<br></div><div>Subject: Re: [Pywps-dev] git branches confusion (mess)<br></div><div>Local Time: August 31, 2017 9:16 AM<br></div><div>UTC Time: August 31, 2017 7:16 AM<br></div><div>From: <a href="mailto:jorge.dejesus@geocat.net">jorge.dejesus@geocat.net</a><br></div><div>To: Jachym Cepicky <<a href="mailto:jachym.cepicky@gmail.com">jachym.cepicky@gmail.com</a>>, <a href="mailto:pywps-dev@lists.osgeo.org">pywps-dev@lists.osgeo.org</a> <<a href="mailto:pywps-dev@lists.osgeo.org">pywps-dev@lists.osgeo.org</a>><br></div><div><br></div><div><br></div><p>Hi to all<br></p><p>I am more in favor of 1 :), but maybe do a pywps 4.0.1 release?<br></p><p>J.<br></p><div class="m_-1635495664983396338moz-cite-prefix">On 28-08-17 07:34, Jachym Cepicky
      wrote:<br></div><blockquote type="cite"><div dir="ltr"><div>Hi all, <br></div><div><br></div><div>seems, we have pushed to `master` instead of to `develop`
          ? <br></div><div><br></div><div>Suggestions? <br></div><div><br></div><div>I would do:<br></div><div><br></div><div>1 - merge master to develop<br></div><div>2 - reset master back to where it was (pywps-4.0.0)<br></div><div><br></div><div>J<br></div></div><div><br></div><div><br></div><pre>_______________________________________________
pywps-dev mailing list
<a class="m_-1635495664983396338moz-txt-link-abbreviated" href="mailto:pywps-dev@lists.osgeo.org">pywps-dev@lists.osgeo.org</a>
<a class="m_-1635495664983396338moz-txt-link-freetext" href="https://lists.osgeo.org/mailman/listinfo/pywps-dev">https://lists.osgeo.org/mailman/listinfo/pywps-dev</a><br></pre></blockquote></blockquote><div><br></div></blockquote></div></div></blockquote><blockquote type="cite"><div><div><span>_______________________________________________</span><br></div><div><span>pywps-dev mailing list</span><br></div><div><span><a href="mailto:pywps-dev@lists.osgeo.org">pywps-dev@lists.osgeo.org</a></span><br></div><div><span><a href="https://lists.osgeo.org/mailman/listinfo/pywps-dev">https://lists.osgeo.org/mailman/listinfo/pywps-dev</a></span><br></div></div></blockquote></blockquote><div><br></div>