Přeskočit obsah

Poskytování recenze žádosti o stažení

Každý je vítán, aby zkontroloval jakýkoli příspěvek k projektu BeeWare. Než začnete, je třeba si uvědomit několik důležitých věcí.

NEŽ NAPÍŠETE RECENZI, ROZMYSLETE SI TO

Než se pustíte do recenze, ZAMYSLETE SE. Jako recenzenti bychom měli zvážit, zda odpověď, kterou se chystáme odeslat, je:

  • Pravda. Vždy se snažte poskytovat přesné návrhy a informace.
  • Užitečné. Poskytujeme pokyny, jak zlepšit podání; tyto pokyny by měly jasně identifikovat zdroj problému nebo nezohledněný případ použití a v ideálním případě poskytnout návod, jak daný problém vyřešit nebo uspokojit dané obavy.
  • Inspirující. Je na nás, abychom autora inspirovali k tomu, aby provedl požadované změny.
  • Nezbytné. Očekává se, že autor si přečte vše, co zveřejníme; musíme respektovat jeho čas a úsilí a zveřejňovat pouze to, co je nezbytné.
  • Laskavost. Stejnou zpětnou vazbu lze vyjádřit mnoha způsoby; musíme se ujistit, že volíme laskavá, podpůrná a konstruktivní slova.

Je zcela možné MYSLET a zároveň poskytovat efektivní recenzi. Výše uvedené koncepty nevylučují poukazování na jakékoli problémy, které v PR najdete. Přispěvatelé nebudou mít příležitost zlepšit svůj příspěvek, pokud si nebudou vědomi oblastí, které je třeba zlepšit. Důležité je mít na paměti, jak tuto zpětnou vazbu prezentujete. Snažte se, aby vaše recenze byla neosobní. Místo „Udělal jste chybu“ můžete říci „Tento kód by mohl být vylepšen“. Recenzujte kód, ne autora.

Je důležité nezapomínat na pozitivní zpětnou vazbu, kromě identifikace oblastí, které je třeba zlepšit. Pokud jsou například změny obzvláště užitečné, dělají něco zvlášť chytrého nebo vám představí API, o kterém jste nevěděli, dejte to autorovi vědět! Nikdy nepodceňujte účinek poukázání na něco, co někdo udělal správně nebo dobře, uprostřed situace, kdy vše ostatní, na co jste poukázali, jsou problémy, které je třeba vyřešit.

Rozhraní GitHub pro revize obsahuje mechanismus pro navrhování změn, ve kterém můžete zadat přesnou změnu, kterou navrhujete jako náhradu za stávající obsah. Mějte na paměti, že dokud nebudou tyto navrhované změny přijaty a potvrzeny, nebudou procházet kontrolami před potvrzením a kontrolami linting. Proto by tato funkce měla být používána pro menší změny, protože čím větší je navrhovaná změna, tím větší je pravděpodobnost, že způsobí problémy.