search

Softcatala / translation-memory-tools

A set of tools to build, maintain and use translation memories
https://www.softcatala.org/recursos/memories/
29 stars 17 forks source link

style: prettify project config files #328

Closed pereorga closed 11 months ago

pereorga commented 11 months ago

Resultat de:

prettier --write cfg/projects/
pereorga commented 11 months ago

@jordimas si ho veus bé després afegeixo un git hook per assegurar que es mantingui la consistència

jordimas commented 11 months ago

Això esta molt bé Pere.

És similar al que fa black que usem pel codi. Més que un git hook jo afegiria un step amb prettier --write al CI com fem amb black: https://github.com/Softcatala/translation-memory-tools/blob/master/.github/workflows/ci.yml#L30 Es podria fer lo del hook, però personalment no em sento confortable amb processos que modifiquin el fitxers al fer commit.

pereorga commented 11 months ago

Em referia a un hook pre-commit amb prettier --check, però puc afegir-ho a la GitHub action (també amb --check) i així no afegim una dependència de desenvolupament.

També penso que té sentit que afegeixi un fitxer package.json amb les ordres check:json i format:json (per exemple) per facilitar-ho. npm continuaria sent opcional.

jordimas commented 11 months ago

Sí, jo faria només el CI/CD (github actions) per no introduir més dependències com dius. De fet n'estic intentant reduint-ne el nombre, com amb a2po que vaig llevar.

pereorga commented 11 months ago

@jordimas a veure com ho veus, crec que és el més estàndard i simple possible. Ho he estès a la resta de fitxers JSON, i hi he incorporat https://github.com/Softcatala/translation-memory-tools/pull/327 (d'on prettier llegeix la configuració, com ho faran els editors).