i18n: Replace toml with json
#20817
i18n: Replace toml with json
#20817
Conversation
|
Find ich ne gute Idee. Einziger Vorteil an TOML find ich cleanere Syntax (keine Klammern und Semicolons). Aber da Weblate das nie als first-class Format mit aufgenommen hat und wir uns da mit |
Da die Dateien nur automatisiert durch Weblate erstellt werden spielt das doch eigentlich keine Rolle, oder? |
für die meisten Leute nicht. Wenn man im UI ein neues Feature baut und neue Keys anlegt schon. Aber wie gesagt, nicht viel :D |
|
Machts denn ein Problem? Deutsche Überschriften sind schon nett solange wir die Tomls re-usen können auch ohne Doppelwartung. Update: ich sehe den Kontext. Warum sollen wir das hier nicht auf Basis json beibehalten? |
|
Sind wir uns denn sicher, dass json keine Probleme macht? Sind insbesondere die Keys sortiert?! Ohne das dürften die Merges ziemlich wild werden. |
Ja, funktioniert im App Repo problemlos. Das konkrete Format heißt i18next JSON v4. Die generierten Translation Files im App Repo haben alle die gleiche Struktur und Sortierung wie die Quelldatei. Keine manuelle Nachbearbeitung oder Escaping/Quoting nötig. |
|
@Maschga hast du ein Migrationsscript was wir einmal kurz vor merge über alle Dateien laufen lassen können? |
|
Ich habe toml-to-json-online-converter genutzt. |
|
Ich habe die Übersetzungsdateien manuell konvertiert, um den PR voranzubringen. Kann sich jemand von euch um den failing Test kümmern? |
|
Im Test scheinen weder die Assets eingebunden noch die live assets verfügbar. Warum ist mir nicht klar. Vmtl. weil das von main aus passiert was im Test nicht zum Einsatz kommt. Da brauchts ne gute Idee. |
|
Gibt es hier PR-seitig noch etwas zu tun? |
|
Der Weblate Teil fehlt? |
|
Ich kümmere mich in den nächsten Tagen um den Rollout. Müssen wir mit Weblate koordinieren und so. |
|
@naltatis @premultiply wie sieht der neue Prozess für Weblate jetzt aus. Können wir PRs wie #20945 jetzt "blind" mergen? |
|
@Maschga es sieht auf jeden Fall super aus- die PRs kommen jetzt sauber und ohne Konflikte. KLASSE! |
|
@andig ja, nur noch auf "groben Unfug" sichten und dann können die rein. Die sollten jetzt keine technische Nachbearbeitung (sort, quote) mehr erfordern. |
Da die
toml-Dateien von Weblate immer wieder für Ärger sorgen und es im evcc-io/app Repo mit denjson-Dateien keine Schwierigkeiten damit gibt, schlage ich vor aufjsonumzusteigen.TODO:
jsonumstellen\cc @naltatis