Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Ithyx
Copy link
Member

@Ithyx Ithyx commented Feb 15, 2024

Description

Cette PR essaye de changer toutes les instances de texte "chanson" en "musique". Le premier commit à été fait avec les commandes suivantes (avec quelques changements exclus, notemment dans la liste des genres):

  • find . -type f -not -path "./node_modules/*" -not -path "./.git/*" -not -path "./.angular/*" -print0 | xargs -0 sed -i '' -e 's/chanson/musique/g'
  • find . -type f -not -path "./node_modules/*" -not -path "./.git/*" -not -path "./.angular/*" -print0 | xargs -0 sed -i '' -e 's/Chanson/Musique/g'

La plupart des instances ont été identifées, mais il est possible qu'il en manque par cause de fautes d'orthographe ou autres.

Checklist

  • Titre
  • Label
  • Catégorie

@Ithyx Ithyx added ui Change user interface minor Increment minor version labels Feb 15, 2024
@Ithyx Ithyx requested review from hvdbm and rakruk February 15, 2024 23:48
@Ithyx Ithyx self-assigned this Feb 15, 2024
Copy link
Member

@rakruk rakruk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ton sed est case sensitive. Il y a des occurrences de Chanson et CHANSON dans le code qui sont pas catch

@Ithyx
Copy link
Member Author

Ithyx commented Feb 16, 2024

Comme mentionné plus haut, Chanson à déjà été pris en compte. J'ai vérifié pour CHANSON mais je n'ai pas trouvé d'instances, c'est dans quel fichier ?

@rakruk
Copy link
Member

rakruk commented Feb 16, 2024

Un exemple ici:

Toutes les occurrences ici:
https://github.com/search?q=repo%3ATableauBits%2FYatga+%22Chanson%22&type=code

Je vois pas tellement d'occurrences de toute manière, je pense que tu peux juste modifier les chansons à la main.

@Ithyx
Copy link
Member Author

Ithyx commented Feb 16, 2024

autant pour moi, j'avais fait une faute dans ma commande, normalement CHANSON est maintenant pris en compte

@rakruk rakruk self-requested a review February 16, 2024 00:27
Copy link
Member

@rakruk rakruk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Member

@hvdbm hvdbm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Ithyx Ithyx marked this pull request as ready for review February 16, 2024 01:35
@Ithyx Ithyx merged commit 10a7bfd into main Feb 16, 2024
@Ithyx Ithyx deleted the maintenance/chanson-to-musique branch February 16, 2024 01:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

minor Increment minor version ui Change user interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants