ktouch sortWords.sh auf neues schema angepasst #596

satanist · 2021-12-31T10:31:24+01:00

satanist commented

2021-12-31 10:31:24 +01:00

nen paar kleine änderungen.

neue dependencies: uuidgen (debian package uuid-runtime), ed

nen paar kleine änderungen. neue dependencies: uuidgen (debian package uuid-runtime), ed

hrnz commented

2021-12-31 20:23:12 +01:00

ich glaube, statische UUIDs wären sinnvoller (damit das Programm beim Importieren einer neueren Version die alte aktualisiert und nicht eine neue hinzufügt).

Auf der anderen Seite ist realistisch gesehen die Warscheinlichkeit, dass das jemals nochmal verändert wird, nicht besonders groß.

LGTM

ich glaube, statische UUIDs wären sinnvoller (damit das Programm beim Importieren einer neueren Version die alte aktualisiert und nicht eine neue hinzufügt). Auf der anderen Seite ist realistisch gesehen die Warscheinlichkeit, dass das jemals nochmal verändert wird, nicht besonders groß. LGTM

satanist commented

2022-01-02 14:47:38 +01:00

Hast du nen vorschlag wie man die uuids statisch erzeugen und dann einlesen kann?

satanist force-pushed ktouch from bb814601e3 to 886f170660

2022-01-08 16:58:42 +01:00

Compare

hrnz commented

2022-01-15 18:38:09 +01:00

Mir fällt da auch nichts besseres ein als sie da einfach hartzukodieren.

Dann hätte man das Problem, das unter Umständen jemand das Skript verwenden möchte, um andere Lektionen aus anderen Inputs zu generieren und das dann die gleiche UUID hat, was auch unvorteilhaft ist.

Ich denke, wir lassen das besser einfach so und lassen dann die Nutzer die alte Version selbst entfernen, falls es jemals eine neue geben wird und jemand dann updaten möchte.

Mir fällt da auch nichts besseres ein als sie da einfach hartzukodieren. Dann hätte man das Problem, das unter Umständen jemand das Skript verwenden möchte, um andere Lektionen aus anderen Inputs zu generieren und das dann die gleiche UUID hat, was auch unvorteilhaft ist. Ich denke, wir lassen das besser einfach so und lassen dann die Nutzer die alte Version selbst entfernen, falls es jemals eine neue geben wird und jemand dann updaten möchte.

hrnz referenced this issue from a commit

2022-01-15 18:38:15 +01:00

Merge pull request 'ktouch sortWords.sh auf neues schema angepasst' (#596) from satanist/neo-layout:ktouch into master

hrnz merged commit 872526210e into master

2022-01-15 18:38:16 +01:00

Sign in to join this conversation.

No reviewers

No Milestone

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: neo/neo-layout#596