Deploying to gh-pages from @ Klipper3d/klipper@1b56a63abf 🚀
This commit is contained in:
@@ -1046,11 +1046,11 @@
|
||||
</li>
|
||||
|
||||
<li class="md-nav__item">
|
||||
<a href="#mire-szamithatunk-egy-felulvizsgalat-soran" class="md-nav__link">
|
||||
Mire számíthatunk egy felülvizsgálat során
|
||||
<a href="#mire-szamithatsz-egy-felulvizsgalat-soran" class="md-nav__link">
|
||||
Mire számíthatsz egy felülvizsgálat során
|
||||
</a>
|
||||
|
||||
<nav class="md-nav" aria-label="Mire számíthatunk egy felülvizsgálat során">
|
||||
<nav class="md-nav" aria-label="Mire számíthatsz egy felülvizsgálat során">
|
||||
<ul class="md-nav__list">
|
||||
|
||||
<li class="md-nav__item">
|
||||
@@ -1310,11 +1310,11 @@
|
||||
</li>
|
||||
|
||||
<li class="md-nav__item">
|
||||
<a href="#mire-szamithatunk-egy-felulvizsgalat-soran" class="md-nav__link">
|
||||
Mire számíthatunk egy felülvizsgálat során
|
||||
<a href="#mire-szamithatsz-egy-felulvizsgalat-soran" class="md-nav__link">
|
||||
Mire számíthatsz egy felülvizsgálat során
|
||||
</a>
|
||||
|
||||
<nav class="md-nav" aria-label="Mire számíthatunk egy felülvizsgálat során">
|
||||
<nav class="md-nav" aria-label="Mire számíthatsz egy felülvizsgálat során">
|
||||
<ul class="md-nav__list">
|
||||
|
||||
<li class="md-nav__item">
|
||||
@@ -1378,7 +1378,7 @@
|
||||
<li>Sikeres felülvizsgálat után a felülvizsgáló "jóváhagyja a felülvizsgálatot" a GitHub-on, és egy <a href="#reviewers">karbantartó</a> átvezeti a módosítást a Klipper master ágába.</li>
|
||||
</ol>
|
||||
<p>Ha fejlesztéseken dolgozol, fontold meg egy téma indítását (vagy a témához való hozzájárulást) a <a href="Contact.html">Klipper Társalgó</a> oldalon. A fórumon folyó vita javíthatja a fejlesztési munka láthatóságát, és vonzhat másokat is, akik érdeklődnek az új munka tesztelése iránt.</p>
|
||||
<h2 id="mire-szamithatunk-egy-felulvizsgalat-soran">Mire számíthatunk egy felülvizsgálat során<a class="headerlink" href="#mire-szamithatunk-egy-felulvizsgalat-soran" title="Permanent link">¶</a></h2>
|
||||
<h2 id="mire-szamithatsz-egy-felulvizsgalat-soran">Mire számíthatsz egy felülvizsgálat során<a class="headerlink" href="#mire-szamithatsz-egy-felulvizsgalat-soran" title="Permanent link">¶</a></h2>
|
||||
<p>A Klipperhez való hozzájárulásokat az egyesítés előtt felülvizsgálják. A felülvizsgálati folyamat elsődleges célja a hibák ellenőrzése, valamint annak ellenőrzése, hogy a beadvány megfelel-e a Klipper dokumentációjában meghatározott irányelveknek.</p>
|
||||
<p>Természetesen egy feladatot sokféleképpen lehet elvégezni; a felülvizsgálat célja nem az, hogy megvitassa a "legjobb" végrehajtást. Ahol lehetséges, a felülvizsgálati megbeszélések inkább a tényekre és a mérésekre összpontosítanak.</p>
|
||||
<p>A beadványok többsége visszajelzést eredményez egy felülvizsgálatról. Készülj fel a visszajelzések beszerzésére, további részletek megadására és szükség esetén a beadvány frissítésére.</p>
|
||||
@@ -1400,7 +1400,7 @@
|
||||
<p>Az előnyök megvitatásakor előnyösebb a "tények és mérések" megvitatása. Általában véve a bírálók nem a "valaki hasznosnak találhatja az X opciót", sem pedig a "ez a beadvány olyan funkciót ad hozzá, amelyet az X firmware valósít meg" formájú válaszokat keresik. Ehelyett általában előnyösebb, ha részletesen tárgyalják, hogy a minőségjavulást hogyan mérték, és milyen eredményeket hoztak ezek a mérések - például: "az Acme X1000 nyomtatókon végzett tesztek a ...képen látható javuló sarkokat mutatnak ", vagy például "az X valós tárgy nyomtatási ideje egy Foomatic X900 nyomtatón 4 óráról 3,5 órára csökkent". Magától értetődik, hogy az ilyen típusú tesztelés jelentős időt és erőfeszítést igényel. A Klipper legjelentősebb funkcióinak némelyike hónapokig tartott a megbeszélések, átdolgozások, tesztelések és dokumentációk során, mielőtt beolvadt a master ágba.</p>
|
||||
<p>Minden új modulnak, konfigurációs opciónak, parancsnak, parancsparaméternek és dokumentumnak "nagy hatással" kell rendelkeznie. Nem akarjuk a felhasználókat olyan opciókkal terhelni, amelyeket nem tudnak ésszerűen konfigurálni, és nem akarjuk őket olyan opciókkal terhelni, amelyek nem nyújtanak számottevő előnyt.</p>
|
||||
<p>A bíráló kérhet pontosítást arról, hogy a felhasználónak hogyan kell beállítania egy opciót - az ideális válasz tartalmazza a folyamat részleteit - például: "a MegaX500 felhasználóinak az X opciót 99,3-ra kell beállítaniuk, míg az Elite100Y felhasználóinak az X opciót a ..." eljárással kell kalibrálniuk...".</p>
|
||||
<p>Ha az opció célja, hogy a kódot modulárisabbá tegye, akkor inkább használj kódkonstansokat a felhasználóval szembenéző konfigurációs opciók helyett.</p>
|
||||
<p>Ha az opció célja, hogy a kódot modulárisabbá tedd, akkor inkább használj kódkonstansokat a felhasználóval szembenéző konfigurációs opciók helyett.</p>
|
||||
<p>Az új modulok, új opciók és új paraméterek nem biztosíthatnak hasonló funkciókat a meglévő modulokhoz - ha a különbségek önkényesek, akkor inkább a meglévő rendszert kell használni, vagy a meglévő kódot kell átalakítani.</p>
|
||||
</li>
|
||||
<li>
|
||||
@@ -1431,13 +1431,13 @@
|
||||
<p>A fehér területi változtatásokat nem szabad összekeverni a funkcionális változtatásokkal. Általánosságban elmondható, hogy az indokolatlan szóköz módosításokat nem fogadjuk el, kivéve, ha azok a módosítandó kód megállapított "tulajdonosától" származnak.</p>
|
||||
</li>
|
||||
</ol>
|
||||
<p>A Klipper nem alkalmaz szigorú "kódolási stílus útmutatót", de a meglévő kód módosításainak követniük kell a meglévő kód magas szintű kódáramlását, kódbehúzási stílusát és formátumát. Az új modulok és rendszerek benyújtása esetén a kódolási stílus rugalmasabb, de előnyösebb, ha az új kód belsőleg konzisztens stílust követ, és általában az iparági kódolási normákat követi.</p>
|
||||
<p>A Klipper nem alkalmaz szigorú "kódolási stílusú útmutatót", de a meglévő kód módosításainak követniük kell a meglévő kód magas szintű kódáramlását, kódbehúzási stílusát és formátumát. Az új modulok és rendszerek benyújtása esetén a kódolási stílus rugalmasabb, de előnyösebb, ha az új kód belsőleg konzisztens stílust követ, és általában az iparági kódolási normákat követi.</p>
|
||||
<p>A felülvizsgálat célja nem a "jobb megvalósítások" megvitatása. Ha azonban egy bírálónak nehézséget okoz egy beadott pályázat megvalósításának megértése, akkor kérhet változtatásokat a megvalósítás átláthatóbbá tétele érdekében. Különösen, ha a bírálók nem tudják meggyőzni magukat arról, hogy egy beadott pályázat hibátlan, akkor változtatásokra lehet szükség.</p>
|
||||
<p>A felülvizsgálat részeként egy felülvizsgáló létrehozhat egy alternatív kérést egy témához. Ezt azért lehet megtenni, hogy elkerülhető legyen a túlzott "oda-vissza" a kisebb eljárási kérdésekben, és így egyszerűsödjön a benyújtási folyamat. Ez azért is megtörténhet, mert a vita arra ösztönzi a bírálót, hogy egy alternatív implementációt készítsen. Mindkét helyzet a felülvizsgálat normális eredménye, és nem tekinthető az eredeti beadvány kritikájának.</p>
|
||||
<h3 id="segitseg-a-felulvizsgalatokban">Segítség a felülvizsgálatokban<a class="headerlink" href="#segitseg-a-felulvizsgalatokban" title="Permanent link">¶</a></h3>
|
||||
<p>Nagyra értékeljük a segítséget a véleményekkel kapcsolatban! Nem szükséges <a href="#reviewers">listázott értékelőnek</a> lenni az értékelés elvégzéséhez. A GitHub kérelmek benyújtóit is arra ösztönözzük, hogy saját beadványaikat vizsgálják felül.</p>
|
||||
<p>A felülvizsgálat során segítsen, kövesse a <a href="#what-to-expect-in-a-review">mire számíthat egy felülvizsgálat során</a> pontban leírt lépéseket a beküldés ellenőrzéséhez. A felülvizsgálat befejezése után adj hozzá egy megjegyzést a GitHub kérésekhez a megállapításokkal. Ha a benyújtás átmegy az ellenőrzésen, akkor kérjük, ezt kifejezetten jelezze a megjegyzésben. Például valami olyasmit, hogy "Átnéztem ezt a módosítást a CONTRIBUTING dokumentumban leírtak szerint, és minden jónak tűnik számomra". Ha nem tudta elvégezni a felülvizsgálat egyes lépéseit, akkor kérjük, kifejezetten jelezze, hogy mely lépéseket vizsgálta felül, és melyeket nem. Például valami olyasmit, mint: "Nem ellenőriztem a kódot hibák szempontjából, de minden mást átnéztem a CONTRIBUTING dokumentumban, és úgy tűnik, hogy minden rendben van".</p>
|
||||
<p>A beadványok tesztelését is értékeljük. Ha a kódot teszteltük, kérjük írjon egy megjegyzést a GitHub kérésekhez a tesztelés eredményével - sikerrel vagy sikertelenséggel. Kérjük, kifejezetten jelezze, hogy a kódot tesztelték, és az eredményeket - például valami olyasmit, mint: "Leteszteltem ezt a kódot az Acme900Z nyomtatómon egy váza nyomtatásával, és az eredmények jók voltak".</p>
|
||||
<p>A felülvizsgálat során segíts, kövesd a <a href="#what-to-expect-in-a-review">mire számíthatsz egy felülvizsgálat során</a> pontban leírt lépéseket a beküldés ellenőrzéséhez. A felülvizsgálat befejezése után adj hozzá egy megjegyzést a GitHub kérésekhez a megállapításokkal. Ha a benyújtás átmegy az ellenőrzésen, akkor kérjük, ezt kifejezetten jelezd a megjegyzésben. Például valami olyasmit, hogy "Átnéztem ezt a módosítást a CONTRIBUTING dokumentumban leírtak szerint, és minden jónak tűnik számomra". Ha nem tudtad elvégezni a felülvizsgálat egyes lépéseit, akkor kérjük, kifejezetten jelezd, hogy mely lépéseket vizsgáltad felül, és melyeket nem. Például valami olyasmit, mint: "Nem ellenőriztem a kódot hibák szempontjából, de minden mást átnéztem a CONTRIBUTING dokumentumban, és úgy tűnik, hogy minden rendben van".</p>
|
||||
<p>A beadványok tesztelését is értékeljük. Ha a kódot teszteltük, kérjük írj egy megjegyzést a GitHub kérésekhez a tesztelés eredményével - sikerrel vagy sikertelenséggel. Kérjük, kifejezetten jelezd, hogy a kódot tesztelted, és az eredményeket - például valami olyasmit, mint: "Leteszteltem ezt a kódot az Acme900Z nyomtatómon egy váza nyomtatásával, és az eredmények jók voltak".</p>
|
||||
<h3 id="ertekelok">Értékelők<a class="headerlink" href="#ertekelok" title="Permanent link">¶</a></h3>
|
||||
<p>A Klipper "értékelők" a következők:</p>
|
||||
<table>
|
||||
|
||||
Reference in New Issue
Block a user