Umstieg auf acro package

RvNovae / dhge-latex

Inoffizelles LaTeX-Template für Projektarbeiten an der Dualen Hochschule Gera Eisenach

8 stars 5 forks source link

Umstieg auf acro package #93

Closed LordofAgents closed 3 years ago

LordofAgents commented 3 years ago

Hallo, hier der PR zum Issue #60 , inklusive aktualisierter ReadMe.md sowie abk.tex. Die Sortierung ist standardmäßig an, das sollte klappen. Das Zählen der Elemente ist noch nicht drin, darum kümmere ich mich jetzt danach. Schaut bitte noch einmal drüber, in meinem Test hat alles geklappt. Danke!

photovoltex commented 3 years ago

die .gitignore bitte noch anpassen :)
Wie sieht es den mit dem automatischem entfernen des Verzeichnissses aus wenn es leer ist? Gibt es da evtl vom Package einen counter den man verwenden kann, oder irgend etwas ähnliches? (ich glaube ein used gesehen zu haben mit dem man nur die verwendeten acros importiert aber ka ob dass die komplette Seite entfernt bzw. nicht schon default an ist)

Und bei mir hat irgendwie link-only-first = false einen Fehler geworfen, nachdem ich es entfernt hatte ging es aber. (das link erstellen macht es bei mir auch bei jedem ohne diesen eintrag) grafik Könnte auch an meiner env liegen (wahrscheinlich, da die ci nicht gemeckert hat), evtl. gibt @RvNovae oder @ZeroPointMax da nochmal Input wenn einer dann dazu kommt sich den mr anzuschauen^^

Wollte auch noch etwas bezüglich Tests sagen, aber das passt ja, da die Vorlage beim testcompile verwendet wird :D

Ansonsten siehts an sich gut aus^^

LordofAgents commented 3 years ago

Ich habs nochmal durchgetestet: Sind deine Pakete aktuell? Das Argument ist relativ neu. Sonst können wir das auch rauslassen, ich sehe gerade in der Doku, das ist standardmäßig auf false.

Das automatische Entfernen habe ich mir noch nicht angeschaut, das steht jetzt auf meiner ToDo.

LordofAgents commented 3 years ago

Bin in den gleichen Fehler gelaufen wie Felix, das Package zu updaten hat das Problem allerdings behoben (vielleicht sollten wir das noch in der readme vermerken?)

Wenn dann würde ich das allgemein recht weit oben vermerken, da es ja nicht nur dieses Paket betreffen kann. Man könnte ja schreiben, dass das Template nur mit den neusten stabilen Paketversionen getestet ist.

RvNovae commented 3 years ago

Bin in den gleichen Fehler gelaufen wie Felix, das Package zu updaten hat das Problem allerdings behoben (vielleicht sollten wir das noch in der readme vermerken?)

Wenn dann würde ich das allgemein recht weit oben vermerken, da es ja nicht nur dieses Paket betreffen kann. Man könnte ja schreiben, dass das Template nur mit den neusten stabilen Paketversionen getestet ist.

Ja genau, vielleicht einfach wie so eine Art FAQ.

RvNovae commented 3 years ago

Ich merge dann jetzt erstmal und kümmer mich dann im Anschluss direkt noch um den Eintrag.

Vielen Dank nochmal für die Contribution!