Closed martinsabo closed 8 years ago
@jsuchal Prestavime code climate alebo lokalneho rubocopa (fail vs raise)?
Ja som zvyknuty na raise
. Inak otazka zni ci chceme hadzat vynimku alebo ju vracat. Ja som zastanca toho, ze vynimky su novodobe goto. ;)
Nieco taketo?
@martinsabo skusme toto trosku otocit, naprv si dohodnime contract, potom to az nabuchaj, lebo si len pridavame zbytocne robotu.
Rozmyslam ake je ocakavane spravanie, ked toto zlyha. Ja by som asi chcel dostat nejaky error v logu cize ta vynimka alebo cokolvek by mala vybublat uplne hore. Notifikacia do slacku ma nezachrani, to si nemusim vsimnut (lebo to realne nemusim ani sledovat). Co myslis ty?
@jsuchal ja netusim ako sa je tam vyrieseny exception logging a pacila sa mi idea, ze aj niekto kto nema pristup k logom uvidi, ze sa nieco deje. Ak teda obsluha slacku nema toto riesit, tak by som to nechal padnut pekne nahlas uz v momente ked sa mu nebude zdat issue page.
Cize:
Ok, tak to nechajme teda tak, ze to hodi exception a zahuci to. Error logger hore by to mal odchytit. Notifikacie to nateraz odchytit asi nemusia.
@jsuchal este nieco tu chyba?
@jsuchal uz by to malo byt ok
Seriozne potrebujeme integracny test. Aspon jeden ;)
On Tue, Feb 23, 2016 at 10:19 AM martinsabo notifications@github.com wrote:
@jsuchal https://github.com/jsuchal uz by to malo byt ok
— Reply to this email directly or view it on GitHub https://github.com/slovensko-digital/uvobot/pull/15#issuecomment-187614137 .
A máš predstavu ako ho urobiť s naším aktuálnym toolsetom?
Od boku ma napadá niečo ako Fake slack api (cez webmock do lokalneho sinatra serveru), Fake discourse api, request na uvo nahraty cez VCR a na tom púšťať uvobot.rb.
Hej toto bude asi jedina cesta. Rozmyslam ci to stoji za to.
Vyspime sa na to :).
:+1:
Pridal som novy typ notification - scraping error. Idea je taka ze postupne tam kde to bude vhodne a potrebne sa moze overovat ci nie su pritomne nejake zmeny/anomalie na UVO strankach ak sa najdu tak to padne so scraping errorom, ktoreho message sa notifikuje na slack.