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

Rend l'argument action insensible à la casse dans la fonction historique #5

Merged
merged 2 commits into from
Feb 22, 2017

Conversation

MatthieuDEVALLE
Copy link
Contributor

Permet d'utiliser:

r.historiques('google')

sans erreur.

Cordialement,

@matthieu637
Copy link
Owner

matthieu637 commented Feb 22, 2017

Il faudrait que ça fonctionne pour tous les caractères.
r.historiques('googlE') aussi

L'idée est bonne, il suffit de l'appliquer entièrement, à la création des clefs aussi.

@MatthieuDEVALLE
Copy link
Contributor Author

Normalement, cela fonctionne correctement.

@matthieu637
Copy link
Owner

Ok je prends déjà ça.

Si tu veux, tu peux essayer de faire une version encore plus générale, sans faire l'hypothèse que la première lettre est en majuscule.
Pour ça, il suffit d'ajouter un key.lower() ligne 191 et modifier un peu ton code.

@matthieu637 matthieu637 merged commit 5c1dc33 into matthieu637:master Feb 22, 2017
matthieu637 pushed a commit that referenced this pull request Mar 2, 2017
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

Successfully merging this pull request may close these issues.

2 participants