search

Pepita73 / webproghu_dev

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

Workflow #4

Closed Endyl closed 6 years ago

Endyl commented 6 years ago

Hogyan megy a fejlesztés, milyen brancheket (vagy branching workflowt) használunk?

Mennyire legyen issue központú a dolog? Pl.: Egy jól definiált issue -> egy branch az erre vonatkozó kóddal -> PR?

Commit üzenetek magyarul vagy angolul? Ezen felül követünk valami formalizmust?

Issue-knak, pull requesteknek legyen formai követelménye? Code of conduct, contributing guildlines? (Lásd Insights > community)

Egyéb?

Pepita73 commented 6 years ago

Egy jól definiált issue -> egy branch az erre vonatkozó kóddal -> PR

Szerintem így.

branch név pl: f_4_workflow ha ez egy feature issue, b_4_workflow, ha ez bugfix. Szerintem így követhető lesz. A "pepita-dev" és hasonló branch-eket kerülném. Issue szerintem assign, aki megoldja (vegyük magunkra, ha elkezdjük).

Commit üzenetek magyarul vagy angolul?

Nekem mindegy, ha csak magyarul tudó fejlesztők dolgoznak rajta, akkor akár magyar is lehet.

Többi kérdésedre válaszként egy kérdés: hogyan tudok olyan jogosultságot adni neked, hogy be tudd állítani, amiket szeretnél / megbeszélünk? :-D Mint mondtam, github adminban nem vagyok otthon, szívesen lepasszolnám. :)

Endyl commented 6 years ago

Elvileg azt tudjuk kezelni, ami a Community részen van :) A settings fül az, amihez csak te férsz hozzá, és nem tudom, hogy ahhoz normál repónál tudsz-e jogosultságot adni. Bár szerintem ezek egyike sem olyan, amiben túl sűrűn kéne matatni. De akkor ha kialakult a megegyezés, és más nem előz meg, akkor beállítom én.

Pepita73 commented 6 years ago

Kérdés: 1. Ha van egy PR és én vagyok a reviewer, ha approve-olom, más is tudja merge-elni? Illetve tőlem függetlenül is tud működni a repó? (Más a reviewer is, más merge) Úgy látom, hogy eléggé különböző időkben érünk rá, és nem szeretném, ha miattam kéne másnak várnia.

2. Teszt. Ez egyben workflow kérdés is, de mi legyen a szabály PR-re? Csak akkor szabad elfogadni, ha a reviewer tesztelte is, vagy ez a beküldő dolga? Annyiból fontos, mert a "munkagépemen" nincs jelenleg Docker, otthonit csak hétvégén használom.

Endyl commented 6 years ago

  1. Nekem az jött le, hogy elég, ha olyan hagyja jóvá, akinek van push joga. Majd meglátjuk a következő PR-nél.

  2. Ha már van review folyamat, használjuk is. Több szem többet lát alapon jobb, ha valaki más is ellenőrzi a változtatásokat. Így még lejjebb szoríthatjuk a "nálam működik" típusú hibákat. Elvégre mindenki tévedhet :)

    Illetve így csökkenthető a code ownership érzése.

ghost commented 6 years ago

Talán nem tartozik annyira szorosan a workflow-hoz, de lesz majd valami automatizált build & deploy, vagy csak copy-paste-elünk majd az éles szerverre, ha kijövünk valami stabil verzióval?

Endyl commented 6 years ago

Szerintem amennyire lehet, automatizálni kéne azt is, mert a copy-paste-ben elég könnyű véletlenül hibázni. Ha nem is feltétlenül ide tartozik, de jó hogy előjött; kéne arra is issue, hogy valaki esetleg lecsaphasson rá :)

Pepita73 commented 6 years ago

9 :)

Pepita73 commented 6 years ago

Labels Felvettem az INFO labelt. Alábbiakban felsorolom, hogy én hogy látom a címkéket, majd ezt szerkesszük szerintem, ahogy csiszolódunk (egy közös látásmódnak kéne kialakulni).

Többi magáért beszél, ha valamelyiket nem használjuk, törölhető szerintem.

Endyl commented 6 years ago

9-ben felmerült, hogy git flow-t, vagy ahhoz hasonló branching modellt használnánk (minimum: fejlesztés develop és arról leágazó brancheken, működő, deployolható verzió pedig visszamerge-ölődik masterbe). Ha teljes egészében git flow szerint mennénk, akkor létezik ez az eszköz is a mindenféle branching műveletek megkönnyítésére. De ennek ellenére mindenkinek át kéne nyálaznia a leírását, aki nem ismeri vagy nem tudja kívülről, hogy tudja mit-hova.

Részemről bármelyik OK.

Endyl commented 6 years ago

Leveszem a doc deeded-et, mert az információk más issuekba kerültek, amik dokumentálva lettek.

ghost commented 6 years ago

Szerintem ez elkészült, esetleg van még bármi egyéb?

Endyl commented 6 years ago

Egyelőre igen; ha lesz még workflow kérdés, majd nyitunk neki másikat.