Closed larstvei closed 1 year ago
Hei igjen og tusen takk for forslaget!
Jeg er en fersk vedlikeholder av dette repositoriumet, men min oppfatning er at lista er intensjonelt sortert i kronologisk rekkefølge med tanke på når en term først ble verifisert. Dersom du ønsker å legge frem et argument for at fila ikke bør sorteres slik, men heller alfabetisk, så hører jeg det gjerne. I så fall skriv helst et nytt issue på dette. Forøvrig kan jeg bemerke at det finnes mange verktøy for å jobbe lokalt med CSV filer som lar deg sortere og filtrere de.
Skjønner!
Jeg tror kanskje det er like greit å lukke denne, siden det er ikke noen veldig sterke argumenter for å holde termene sortert (det er jo som du sier veldig lett å sortere dem om det trengs). Da jeg åpnet filen ble jeg bare overrasket over at ordene ikke var sortert, og tenkte ikke over at det kunne være kronologisk.
I prosessen merket jeg meg at ordlisten ikke er godt sortert på nettsiden heller. For eksempel kommer å
før ø
. Jeg skal åpne et issue på det.
Takk for at du nevner det! Jeg la merke til det selv også her om dagen, men glemte å notere det. Uansett, da lukkes denne innhentingsforespørselen frem til videre.
Hei!
Med denne endringen vil ordene i
verifiserte_termer.csv
være sortert. I tillegg er det lagt til en test som sjekker at termene er sortert itest.py
.Det er ikke en spesielt vesentlig endring, men det gjør det litt enklere å jobbe med filen. Eksempelet som motiverte endringen er at jeg har en ordliste på et par hundre ord, hvor jeg har lyst til å raskt kunne filtrere ut de ordene som er i ordlisten fra før.
Merk at sorteringen ikke skiller mellom store og små bokstaver (som ser ut til å være tilfelle i sorteringen på nettsiden også).