Skip to content
This repository has been archived by the owner on Jul 9, 2023. It is now read-only.

TEE #390

Merged
merged 61 commits into from
Jul 26, 2022
Merged

TEE #390

merged 61 commits into from
Jul 26, 2022

Conversation

embix
Copy link
Contributor

@embix embix commented Jul 25, 2022

will solve #389 eventually

embix and others added 25 commits July 25, 2022 12:12
- BR 187 um DACH-Zulassung und Last Mile ergänzt
* Add stations on Paris-Lille route

* Add classique line Paris-Lille

* Add Long... Boulogne stations

* Remove/replace Long...

* Add Route Amiens-Calais

* Shift coordinates

* Fixes, shifted stations, etc.

* Update pathSuggestions

* Add Lille-Calais

* Add XFHZ-XFDK
* AT Tasks

* detailed Mittenwaldbahn

* Projected coordinates

Co-authored-by: TrainCompany <train-company@c1710.de>
* Add Görlitz-Dresden

* Projected coordinates

Co-authored-by: TrainCompany <train-company@c1710.de>
marhei and others added 18 commits July 26, 2022 17:54
- BR 187 um DACH-Zulassung und Last Mile ergänzt
* Add stations on Paris-Lille route

* Add classique line Paris-Lille

* Add Long... Boulogne stations

* Remove/replace Long...

* Add Route Amiens-Calais

* Shift coordinates

* Fixes, shifted stations, etc.

* Update pathSuggestions

* Add Lille-Calais

* Add XFHZ-XFDK
* AT Tasks

* detailed Mittenwaldbahn

* Projected coordinates

Co-authored-by: TrainCompany <train-company@c1710.de>
* Add Görlitz-Dresden

* Projected coordinates

Co-authored-by: TrainCompany <train-company@c1710.de>
* ICNG

Könnte man es so hinzufügen?

* Verschiebung der niederländischen Elektrotriebzüge

Ich habe die niederländischen Elektrotriebzüge zu den restlichen Elektrotriebzügen verschoben. Ich habe mich ehrlich gesagt schon gefragt, warum die sich zwischen den Dieseltriebzügen befanden.

FYI: I hate merge conflicts!
(cherry picked from commit 9365303)
* Update issue templates

(cherry picked from commit 9365303)

* Don't use checkboxes for now, as they are presented as subtasks
@embix
Copy link
Contributor Author

embix commented Jul 26, 2022

only 40% of #389 - but should probably be merged already in order to avoid further merge conflicts

@embix embix marked this pull request as ready for review July 26, 2022 16:19
@embix
Copy link
Contributor Author

embix commented Jul 26, 2022

Wo Konflikt? 🤨

@marhei
Copy link
Owner

marhei commented Jul 26, 2022

Ich glaube du musst dir erstmal die aktuelle Version von den Daten auf dein Fork holen.

@embix
Copy link
Contributor Author

embix commented Jul 26, 2022

Ich glaube du musst dir erstmal die aktuelle Version von den Daten auf dein Fork holen.

Hatte ich ja - und anschließend erstmal Konfliktbeseitigung gemacht 😐
Und danach hat er den neuen Part als Konflikt erkannt - warum auch immer. Sinnloser Squashmerge/Remerge hat immerhin geholfen.

@embix
Copy link
Contributor Author

embix commented Jul 26, 2022

Merge ging erst nicht ordentlich (hab hier aktuell nur winmerge - damit komm ich nicht so klar, auf Arbeit nehme ich eher p4merge oder gmaster), deswegen hab ich dann master auf meinen fork rebased und die Einzelcommits fit gemacht. Das ist etwas weniger anstrengend als alles auf einmal.

Nun sollte es aber passen.

@marhei marhei merged commit 5292829 into marhei:main Jul 26, 2022
@embix embix deleted the tee-classic branch July 26, 2022 17:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants