Creation d'une seule RegEx principale pour 'Manu'#22
Open
clementFoyer wants to merge 3 commits intoBo-Duke:masterfrom
Open
Creation d'une seule RegEx principale pour 'Manu'#22clementFoyer wants to merge 3 commits intoBo-Duke:masterfrom
clementFoyer wants to merge 3 commits intoBo-Duke:masterfrom
Conversation
Le remplacement par le mot 'Manu' se fait pour la majorité des références au président de la république française (ou toutes les formules raccourcies aussi), avec ou sans accent, avec ou sans cédille. Prise en compte des versions raccourcies de "monsieur" aussi. Ajout de deux formules anglophones. Ça pourrait être enrichi ultérieurement avec d'autre langues. Prise en compte de l'apostrophe (à la place de la simple quote qui est toujours présente, mais sans doute un peu inutile ici). Prise en compte de l'issue Bo-Duke#1 avec la prise en compte du couple. Ici double référence, en disant bien "couple *à*" en référence au tollé de la "fête à macron" ; on conserve la notion de couple plutôt que de remplacer par juste "Manu" qui exclu Brigitte (qui n'a rien demandé, la pauvre), et si on remplaçait par "Manu et Brigitte", il aurait fallu accorder les verbes en conséquence, et c'est pas le but ici. Ne pas remplacer lors d'un président de parti/région/asso En partie en raison de la page indiquée dans l'issue#2 où était indiqué "le président (LR) de ...". Prise en compte un peu plus propre des apostrophes pour d' et qu'. Corrige l'issue Bo-Duke#21. Corrige l'issue Bo-Duke#6. retirer la dernière condition de changement sur juste "Macron" pour eviter que "Brigitte Macron" ne devienne "Brigitte Manu". Ça manque de negative look-behind pour faire ça bien je crois... La sous string extraite permet de réutiliser la string pour la seconde regexp utile pour détecter les formes telles que "du président" tel que mentionné dans la PR Bo-Duke#12 Ajout de check de bornes de mots (notamment important pour les "de" "que" ou "du".
1f45583 to
7fd57d4
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Cette PR remplace la PR #8 avec une version plus lisible, et plus simple à comprendre pour qui n'est pas habitué aux Regex. Il ne reste quelques unes, il reste deux trois subtilités, mais tout est commenté :)
Le remplacement par le mot 'Manu' se fait pour la majorité des
références au président de la république française (ou toutes les
formules raccourcies aussi), avec ou sans accent, avec ou sans
cédille. Prise en compte des versions raccourcies de "monsieur" aussi.
Ajout de deux formules anglophones. Ça pourrait être enrichi
ultérieurement avec d'autre langues.
Prise en compte de l'apostrophe (à la place de la simple quote qui est
toujours présente, mais sans doute un peu inutile ici).
Prise en compte de l'issue qui existait qui demandait la prise en compte du couple. Ici
double référence, en disant bien "couple à" en référence au tollé de
la "fête à macron" ; on conserve la notion de couple plutôt que de
remplacer par juste "Manu" qui exclu Brigitte (qui n'a rien demandé, la
pauvre), et si on remplaçait par "Manu et Brigitte", il aurait fallu
accorder les verbes en conséquence, et c'est pas le but ici.
Ne pas remplacer lors d'un président de parti/région/asso
En partie en raison de la page indiquée dans l'issue#2 où était indiqué
"le président (LR) de ...".
Prise en compte un peu plus propre des apostrophes pour d' et qu'.
Corrige l'issue #21 et #23 .
Corrige l'issue #6.
retirer la dernière condition de changement sur juste "Macron" pour
eviter que "Brigitte Macron" ne devienne "Brigitte Manu". Ça manque de
negative look-behind pour faire ça bien je crois...
La sous string extraite permet de réutiliser la string pour la seconde
regexp utile pour détecter les formes telles que "du président" tel que
mentionné dans la PR #12
Ajout de check de bornes de mots (notamment important pour les "de"
"que" ou "du".