Zápis 26.11.2018 09:00 - 12:00

Miestnosť: 4.03

Vedúci: Ing. Jakub Šimko, PhD.

Zapisovateľ: Bc. Michal Fabiš

Scrum master: Bc. Martin Žák

Prítomní:

• Ing. Jakub Šimko, PhD.
• Bc. Martin Žák
• Bc. Katarína Rafčíková
• Bc. Andrej Slaninka
• Bc. Daniela Sitárová
• Bc. Andrej Zaťko
• Bc. Michal Fabiš
• Bc. Maroš Vašš

Komentáre v kóde - Ako? Kedy? Skôr NIE!

• Nové metódy (ale aj celkovo) ktoré treba modifikovať má zmysel komentovať iba v prípade že, pridávam informáciu navyše oproti signatúre kódu. Teda keď máme dobre spravenú hlavičku a sú jasné všetky parametre tak nebudem komentovať aj tie ktoré sú jasné.
• Pokiaľ sa v kóde volá metóda mimo kontextu, tak je to ťažké, najdôležitejšie je aby ju bolo jednoduché pochopiť v mieste volania. V komentári sa nemajú vyskytovať veci ktoré sa môžu vyskytovať v signatúra.
• Rule of thumb je že názov metódy je tak max z troch slov, názvy parametrov tak z dvoch slov.
• Má zmysel to komentovať naprv ak treba ten parameter upresniť, napr. že hodnota môže byť od tolko do tolko, inak to nebude fungoať.
• Všeobecný pokec by mal obsahovať čo? Musí to byť informácia navyše a musí to byť dosť HIGH level aby vydržal zmeny.
• Teraz nerobíme API, takže to nebude nikto iný používať - vidíme implementáciu

TODO

• Exportovanie úloh z JIRA každý týždeň nám chýba. Treba urobiť export úloh/šprintov, kvôli komisií a pod. KAŽDÝ TÝŽDEŇ - Za úlohu si berie Danka
• Zavesiť dokumentáciu na WEB. - AndyS
• Pozrieť nástroj na generáciu dokumentácie zo zdrojového kódu (Angular a NodeJS) - Mišo - task za 0 bodov na ďalší šprint, dať aj do dependencies
• Zmeniť metodiku - tak aby nás neobmedzovala - že keď je nutkanie tak môžeme. Komentár metódy má obsahovať iba informácie ktoré dopĺňajú signatúru metódy (vstupy, výstupy, telo) - Do šprintu
• Napísať metodiku písania zdrojového kódu - upraviť tak aby sedela s komentovaním.
• Poslať dokumentáciu JŠ. - Maťo
• Dneska zbuildovať prehliadač - aby sme ho vedeli normálne používať - Maťo.
• Rozdeliť storku WEB132 - s Rozdeleím skrývania hlavného webview tagu - dvaja ludia max nech sa na to pozrú a rozdelia to - AndyS, AndyZ
• Pozrieť sa na chybu WEB133 - pa11y ďo ďalšieho pondelka - Danka
• Ďalší šprint môžeme robiť interné testovanie - Webable tím members