Added Ufa, Kyiv, Rezkene und Ventspils

[embix]

Kannst du demnächst bei deinen Commitmessages mal -v machen (aka: etwas ausführlicher / verbose), damit man zumindest ne grobe Ahnung, was der patch macht?

Deine Commits/Files changed sind so groß, daß github sich sträubt ein diff anzuzeigen:

14,699 additions, 9,765 deletions not shown because the diff is too large. Please use a local Git client to view these changes.

So daß man gezwungen wäre, deinen branch/pr lokal extra zu fetchen und in gitk o.ä. aufzumachen um zu sehen was passiert.

Es ist auch schöner wenn in git -log oder gitk zu jedem commit eine kurze Zusammenfassung in der ersten Zeile steht Added lots of lines) ist da weniger hilfreich. So was wie added routes in $region/$country wäre schöner, also konkret added routes in US midwest oder so. Wenn es viele sind (so wie hier), kann man die auch in der commitmessage auflisten - oder noch besser: für jede Route einen eigenen Commit. Das würde auch Debugging einfacher machen (Stichwort bisect).

Das fixed stuff kann man gerade noch durchgehen lassen, da der diff so klein ist, daß man es zumindest auf den ersten blick sieht.

Aside: Zur besseren Abgrenzung würde ich line für Eisenbahnlinien im Sinne von eine (regelmäßige/getaktete) Bahnverbindung verstehen, also sowas wie eine RB- oder S-Bahnlinie, route für Route/Strecke.

MVGFan8xx:ichhabekeineahnungmehrwasichdashiernennensoll

Ich nehme für sowas gerne einen "sprechenden Namen", z.b. add-route-rumburk-decin wo dann i.d.R. stations, route und evtl. passende jobs in mehreren Commits zu einem "Themen"/Topic branch drin sind.

[MVGFan8xx]

@embix Okay, ist notiert. Danke für den Hinweis. Ich werds in Zukunft beachten!

[marhei]

Kannst du bitte als nächstes den Knoten Rund um Moskau entwirren? Da kreuzen aktuell mehrere Bahnstrecken ohne Verbindung, die in echt existiert…

[marhei]

Allgemein müsste in dem neuen Netz etwas aufgeräumt werden.