Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Uniformisation des fichiers #136

Open
sbrodin opened this issue Feb 11, 2020 · 9 comments
Open

Uniformisation des fichiers #136

sbrodin opened this issue Feb 11, 2020 · 9 comments

Comments

@sbrodin
Copy link
Contributor

sbrodin commented Feb 11, 2020

Je trouve que les noms des fichiers dans sounds sont un peu aléatoirement définis (underscore, tiret, majuscules, nom partiel ou complet de la réplique, etc.)
Est-ce qu'il ne faudrait pas uniformiser tout ça, auquel cas il faudrait se mettre d'accord sur une norme à adopter ?

@Aluriak
Copy link
Contributor

Aluriak commented Feb 11, 2020

Bonne idée. Ça peut se faire avec un petit code python. Ça m'occupera dans les jours prochains, tiens.

Je propose d'utiliser le format suivant : CamelCasePourLesNomsDeFichiersOuSontEnlevesLAccentEtLApostrophe.mp3

Pour effectuer la correction, je me propose de faire le code et le commit, une fois que le format sera déterminé par les contributeurs.

@sbrodin
Copy link
Contributor Author

sbrodin commented Feb 11, 2020

Je trouvais ça plus lisible en mettant des tirets ou des underscores. Et tout en minuscule et sans accent évidemment.
Après il faut s'entendre pour tous les remplacements d'apostrophes ("j'ai" ou "c'est" par exemple).
Et puis il faut faire quelque chose pour les fichiers qui peuvent avoir le même nom ("HÉRÉTIQUE ! AU BUCHER !" notamment).

@2ec0b4
Copy link
Owner

2ec0b4 commented Feb 11, 2020

J'avoue, je suis assez permissif sur le format des noms des fichiers.

Quand j'ai vu passer cette issue, j'ai pensé à @Aluriak et son dépôt https://github.com/Aluriak/kaamelott-soundboard-viz 😊

Pour effectuer la correction, je me propose de faire le code et le commit, une fois que le format sera déterminé par les contributeurs.

🙏

Je trouvais ça plus lisible en mettant des tirets ou des underscores. Et tout en minuscule et sans accent évidemment.

J'ai une préférence pour ce format également, en choisissant les tirets. D'autant que ça apparait dans l'URL. D'ailleurs, il y aura certainement une gestion de redirection à faire, si on veut faire ça bien 🤔

@2ec0b4
Copy link
Owner

2ec0b4 commented Feb 11, 2020

@Aluriak
Copy link
Contributor

Aluriak commented Feb 13, 2020

AJA Kebab case, que j'utilisais sans nommer pendant ma période lisp.

Donc on partirais sur du kebab-case-ie-tiret-comme-separateur-et-pas-daccent-ni-apostrophe.mp3 ?

@Aluriak
Copy link
Contributor

Aluriak commented Feb 13, 2020

Quand j'ai vu passer cette issue, j'ai pensé à @Aluriak et son dépôt https://github.com/Aluriak/kaamelott-soundboard-viz blush

Oui, va falloir que j'y rejette un œil. J'avais fait un truc qui réordonnait proprement le json, aussi.

@Aluriak
Copy link
Contributor

Aluriak commented Feb 13, 2020

Après il faut s'entendre pour tous les remplacements d'apostrophes ("j'ai" ou "c'est" par exemple).

Je propose de les enlever, comme les accents.

Et puis il faut faire quelque chose pour les fichiers qui peuvent avoir le même nom ("HÉRÉTIQUE ! AU BUCHER !" notamment).

C'est le plus chiant. Je dirais bien de suffixer par -2.

@2ec0b4
Copy link
Owner

2ec0b4 commented Feb 13, 2020

AJA Kebab case, que j'utilisais sans nommer pendant ma période lisp.

Donc on partirais sur du kebab-case-ie-tiret-comme-separateur-et-pas-daccent-ni-apostrophe.mp3 ?

👍

Oui, va falloir que j'y rejette un œil. J'avais fait un truc qui réordonnait proprement le json, aussi.

Cf. #115

J'avais dit que j'allais mettre ça dans un crochet (hook git) mais je ne l'ai pas fait 😅

C'est le plus chiant. Je dirais bien de suffixer par -2.

Alors, je me pose la question suivante : est-ce qu'il faut mettre -1 aussi du coup ? 🤔🙃

@Aluriak
Copy link
Contributor

Aluriak commented Feb 14, 2020

Nop, je pense pas. Inutile, d'autant que ça ne change pas leur ordre lexicographiquement. Bref, c'est la même chose fonctionnellement, et c'est IMHO plus clair.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants