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

Suppression des warnings "Lambda may not be necessary" #2541

Merged
merged 1 commit into from Apr 16, 2015

Conversation

Projects
None yet
4 participants
@francoisthire
Contributor

francoisthire commented Apr 14, 2015

Q R
Correction de bugs ? non
Nouvelle Fonctionnalité ? non
Tickets (issues) concernés

Cette PR corrige tous les warnings "Lambda may not be necessary" (ou bien de façon théorique les Eta-conversion.

Pour la QA, il n'y a pas grand chose à vérifier si ce n'est que je n'ai pas changé la sémantique du code en faisant par exemple un :
python manage.py test zds

@Eskimon

This comment has been minimized.

Show comment
Hide comment
@Eskimon

Eskimon Apr 14, 2015

Member

AMHA Travis suffira ici :)

Member

Eskimon commented Apr 14, 2015

AMHA Travis suffira ici :)

@landscape-bot

This comment has been minimized.

Show comment
Hide comment
@landscape-bot

landscape-bot Apr 14, 2015

Code Health
Repository health increased by 0.63% when pulling bb99b55 on Saroupille:review into dcc9e38 on zestedesavoir:dev.

landscape-bot commented Apr 14, 2015

Code Health
Repository health increased by 0.63% when pulling bb99b55 on Saroupille:review into dcc9e38 on zestedesavoir:dev.

@Eskimon

This comment has been minimized.

Show comment
Hide comment
@Eskimon

Eskimon Apr 14, 2015

Member

Bon, on a le vieux problème de Travis qui sera bientot réglé, en attendant difficile de conclure si tout va bien

Member

Eskimon commented Apr 14, 2015

Bon, on a le vieux problème de Travis qui sera bientot réglé, en attendant difficile de conclure si tout va bien

@Eskimon

This comment has been minimized.

Show comment
Hide comment
@Eskimon

Eskimon Apr 14, 2015

Member

Yes mon relancement de Travis a fait passer les choses :) Du coup si @SpaceFox ou @firm1 peut faire une code review je pense qu'on sera bon pour merge :)

Member

Eskimon commented Apr 14, 2015

Yes mon relancement de Travis a fait passer les choses :) Du coup si @SpaceFox ou @firm1 peut faire une code review je pense qu'on sera bon pour merge :)

@pierre-24

This comment has been minimized.

Show comment
Hide comment
@pierre-24

pierre-24 Apr 15, 2015

Member

Moi je suis d'accord avec la correction, si jamais. On conserve bien le but de base, à savoir des noms tout différents et qui se suivent au numéro.

Member

pierre-24 commented Apr 15, 2015

Moi je suis d'accord avec la correction, si jamais. On conserve bien le but de base, à savoir des noms tout différents et qui se suivent au numéro.

@Eskimon

This comment has been minimized.

Show comment
Hide comment
@Eskimon

Eskimon Apr 16, 2015

Member

Ok je merge alors (et comme les factory sont beaucoup utilisée dans les TU et que Travis est heureux, ca me semble de toute facon "self-approved" :D ) !

Member

Eskimon commented Apr 16, 2015

Ok je merge alors (et comme les factory sont beaucoup utilisée dans les TU et que Travis est heureux, ca me semble de toute facon "self-approved" :D ) !

Eskimon added a commit that referenced this pull request Apr 16, 2015

Merge pull request #2541 from Saroupille/review
Suppression des warnings "Lambda may not be necessary"

@Eskimon Eskimon merged commit 808bd99 into zestedesavoir:dev Apr 16, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@Eskimon Eskimon added this to the Version 1.8 milestone Apr 16, 2015

@Eskimon Eskimon added Evolution C-Back and removed Evolution labels Apr 16, 2015

@Eskimon

This comment has been minimized.

Show comment
Hide comment
@Eskimon

Eskimon Apr 16, 2015

Member

Et merci Saroupille !!

Member

Eskimon commented Apr 16, 2015

Et merci Saroupille !!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment