refactor(terminsuche): Consistently use and log exceptions #127
Conversation
haslersn
force-pushed
the
its-error-handling
branch
2 times, most recently
from
84a1659
to
4dd6768
Compare
haslersn
force-pushed
the
its-error-handling
branch
2 times, most recently
from
e5a5782
to
05c5971
Compare
|
Hey, bei diesem Projekt haben wir uns explizit dafür entschieden, deutsche Begriffe für Methoden und Variablen zu nutzen sowie die Kommentierungen in Deutsch vorzunehmen. Das ist total unüblich und fühlt sich auch irgendwie falsch an, aber es soll der Verständlichkeit dienen, wenn sich mal jemand den Code anschaut und schnell verstehen möchte, was da genau passiert. Durch den ganzen Zuwachs ist das nicht mehr ganz einheitlich und an manchen Stellen geht das auch nicht, aber es wäre cool, wenn wir weitestgehend dabei bleiben könnten. Würdest du das entsprechend anpassen? Danke. |
|
Ja. Edit: Andererseits sind alle Methoden- und Attributnamen aus Libraries (z. B. Python-Standardbibliothek) dennoch auf englisch. Reicht es nicht den Output auf Deutsch zu machen? |
haslersn
force-pushed
the
its-error-handling
branch
2 times, most recently
from
f5d7dc1
to
cbbd221
Compare
|
@haslersn bei Imports lässt es sich nicht vermeiden und ich finde es wichtig, dass wir einheitlich bleiben. Begriffe wie "driver", etc. können englisch bleiben, aber ich sehe den Mehrwert nicht, jetzt alle Bezeichnungen im Programm umzuändern. |
Before this commit, `book_appointment`/`backup_book_appointment` could return `True`, return `False` or raise an exception. Now we simply always raise an exception if something goes wrong. This commit also pushes Success and error logging down the call stack. The error message for logging is taken from the exception. This way, every exception is logged consistently.
haslersn
force-pushed
the
its-error-handling
branch
from
cbbd221
to
22f08c6
Compare
|
passt für mich, kann das gemerged werden @haslersn? der PR war eben noch auf "not ready" und ich hab ihn ausversehen auf "ready for review" gesetzt. |
|
Ne, der ist not ready, weil es noch Probleme im Zusammenspiel mit dem retry decorator gibt. |
|
Schließe, da kaum kompatibel mit #351. |
Before this commit,
book_appointment/backup_book_appointmentcould returnTrue, returnFalseor raise an exception. Now we simply always raise an exception if something goes wrong. This commit also pushes Success and error logging down the call stack. The error message for logging is taken from the exception. This way, every exception is logged consistently.