More precision and added Brigitte #12
Open
Conversation
This file contains 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
Added a better precision with space checking on the name (for @EmmanuelMacron on twitter), took Brigitte into account too.
|
looks like it will not work for a sentence like "la maison DU Président de la République". |
clementFoyer
added a commit
to clementFoyer/Manu-Auto-Correct
that referenced
this pull request
Jun 22, 2018
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 "M." ou "Mr".
clementFoyer
added a commit
to clementFoyer/Manu-Auto-Correct
that referenced
this pull request
Jun 23, 2018
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 "M." ou "Mr".
clementFoyer
added a commit
to clementFoyer/Manu-Auto-Correct
that referenced
this pull request
Jun 26, 2018
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".
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.
Added a better precision with space checking on the name (for @EmmanuelMacron on twitter),
took Brigitte into account too.