Pepita73 / webproghu_dev

Webprog.hu apache-php7.2, Drupal 8.5.5
1 stars 1 forks source link

develop branch #20

Closed Endyl closed 6 years ago

Endyl commented 6 years ago

17 alapján létre kell hozni a develop branchet, és beállítani a Settings > Branches részen egy szabályt, hogy oda is kelljen PR és legalább 1 review. Meg az új commit utáni review elavulást is be lehet állítani szerintem.

@Pepita73: neked adtam, mert a beállításokhoz csak te férsz hozzá.

Pepita73 commented 6 years ago

ok pár óra múlva csinálom

Pepita73 commented 6 years ago

Jelenleg ez van, a két rule egyforma (csak az 1 db review van bepipálva) image

Pepita73 commented 6 years ago

Kérdés: mit jelent a github-on a default branch? Ha ugyanazt, mint gitLAB-on, akkor ez egy kicsit kényelmetlen / odafigyelős lesz itt is. Mert a default-ból szeretné létrehozni a gyorsmenüben: image

Rá kell állni előbb a develop-ra, utána ajánlja abból az újat. Erre figyelni kell, bár elvileg sok bajunk nem lesz véletlenszerű master -> develop merge-ből, legfeljebb ha hotfix-nél elfelejtettük, akkor így pótlodik. De ha van valami jó trükk erre, osszuk meg.

Pepita73 commented 6 years ago

Bocsi, most néztem, hogy kell az elavulás is, szóval most ilyen mindkettő: image

Endyl commented 6 years ago

Nem kell githubon branchet készíteni :D

Ezért mondtam a git flow szerű eszközöket, mert azok elvileg megoldják a megfelelő helyről történő leágazást és a megfelelő helyre történő merge-ölést.

Pepita73 commented 6 years ago

@Endyl , kérlek check után zárd le, ha megfelel.

Pepita73 commented 6 years ago

Hááát én inkább szeretem a parancssort, illetve 1-2 sűrűn előforduló több parancsos műveletre vagy feledékenység ellen a bash scriptet. :)

Endyl commented 6 years ago

A git flow parancssoros cucc :)

ghost commented 6 years ago

@Endyl Itt mi volt a dokumentálandó információ? Lehet csak véletlenül lett doc needed?

Endyl commented 6 years ago

Szerintem az, hogy mi, honnan ágazik le és hova tér vissza. Az meg benne van a branching megtárgyalásában és a wikiben is. Leveszem róla, és szerintem documented sem kell rá.