search

bablokb / mtv_cli

Mediathekview auf der Kommandozeile
GNU General Public License v3.0
21 stars 8 forks source link

Formatierung #8

Closed MaxG87 closed 3 years ago

MaxG87 commented 3 years ago

Dieser PR baut auf #7 auf. Der Quellcode wird mittels automatischer Tools neu formatiert. Da nun die Formatierung und Importstrukturierung gänzlich den Programmen überlassen werden kann,

Dieser PR soll bitte als Vorschlag verstanden werden. Es ist mir durchaus aufgefallen, dass der Quellcode mit viel Sorgfalt von Hand formatiert wurde und ich kann nachvollziehen, wenn dies nicht aufgegeben werden soll.

bablokb commented 3 years ago

Ich würde einen Teil der Commits gerne übernehmen, aber nicht den kompletten Pull-Request. Hintergrund ist die Einrückung auf vier Stellen. Das ist zwar im Pythonumfeld der Standard, aber ich empfinde das nicht nur als Platzverschwendung: schon mit einer Einrücktiefe von 2 werden Zeilen oft recht lang. Ich versuche sie auf ca. 80 Zeichen zu begrenzen (mehr ist unergonomisch) und eine größere Einrücktiefe zwingt dann dazu, mehr Befehle auf mehrere Zeilen zu verteilen, was die Lesbarkeit auch nicht gerade erhöht. Aber das ist Geschmackssache. Genauso wie die Verteilung der Imports: einen pro Zeile halte ich für eine unnötige Aufblähung des Codes, ich gruppiere hier gerne.

MaxG87 commented 3 years ago

Ich werde den PR nochmal neu erstellen. Formatierungsänderungen verursachen zu viele Mergekonflikte, um den PR später zu verwenden.