search

nerrixde / DSBApi

API für die in Deutschland verbreitete DSBmobile App für Vertretungspläne an Bildungseinrichtungen
GNU General Public License v3.0
35 stars 6 forks source link

fetch_api() lesbarer und aufgeräumter gemacht #4

Closed kangalio closed 4 years ago

kangalio commented 4 years ago

Die Logik ist gleich geblieben, aber ich habe Kommentare und Absätze eingefügt, sowie verschachelte Statements aufgebrochen und lesbar gemacht.

nerrixde commented 4 years ago

Ich werde es erstmal in dev mergen, da ich es erst am PC testen muss.

kangalio commented 4 years ago

Ich hatte den Commit noch nicht gepusht, in dem ich DATA_URL in den Konstruktur verschoben habe. Das müsstest du dann selbst noch hinzufügen, weil du schon gemergt hast

nerrixde commented 4 years ago

Mach mal n neuen pr

kangalio commented 4 years ago

Ich hatte den Commit noch nicht gepusht, in dem ich DATA_URL in den Konstruktur verschoben habe. Das müsstest du dann selbst noch hinzufügen, weil du schon gemergt hast

Du meinst ich soll eine neue Pull Request dafür machen?

nerrixde commented 4 years ago

yep

kangalio commented 4 years ago

Äh.. aber du kannst das doch auch selber machen? Das ist doch viel zu viel Aufwand, eine neue Pull Request zu machen, nur um zwei Zeilen Code zu verändern. Und es wäre auf jeden Fall einfacher für dich gewesen, die triviale Änderung selbst vorzunehmen, als:

  1. Einen Branch erstellen
  2. Im neuen Branch meine Pull Request rückgängig machen
  3. Branch mergen
  4. Mich um neue Pull Request bitten
  5. Pull Request reviewen
  6. Pull Request mergen

Ich will nicht unfreundlich wirken, aber das wirkt etwas seltsam

nerrixde commented 4 years ago

Ja, ich hätte es auch rebasen können, hast recht.