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

Nettoyage des MP #2207

Merged
merged 6 commits into from Feb 4, 2015

Conversation

@gustavi
Copy link
Member

commented Feb 4, 2015

Reprise de #1993

Q R
Correction de bugs ? non
Nouvelle Fonctionnalité ? non
Tickets concernés x

Dans cette PR j'ai essayé de nettoyer un peu certain fichiers des MP. Ça comprend :

  • l'ajout de commentaires ;
  • l'amélioration de l'interface d'administration ;
  • la suppression d'une fonction jamais utilisée.

Toutes les remarques sont les bienvenues !

QA

  • Vérifier si tous les MP fonctionne encore
  • Vérifier que l'interface d'admin des MP fonctionne encore
  • Vérifier la pertinence et l'orthographe des commentaires

C'est juste un rebase, ça a déjà passé la QA

@gustavi gustavi referenced this pull request Feb 4, 2015
@Eskimon

This comment has been minimized.

Copy link
Member

commented Feb 4, 2015

Rebase sans conflit comme j'ai fait hier ? Du coup je peux merger quand l'ami Travis sera content ?

@Eskimon

This comment has been minimized.

Copy link
Member

commented Feb 4, 2015

Ah bah non, fail mystique de Travis... Je relance ou ca sert a rien ? (ping @SpaceFox si tu veux analyser...)

@GerardPaligot

This comment has been minimized.

Copy link
Member

commented Feb 4, 2015

L'ami Travis ne sera pas content tant que cette issue ne sera pas résolue.

@SpaceFox

This comment has been minimized.

Copy link
Member

commented Feb 4, 2015

Oui, s'il n'y a que les 2 erreurs mentionnées dans le #2202, on peut considérer que Travis est passé.

@Eskimon

This comment has been minimized.

Copy link
Member

commented Feb 4, 2015

Okai, on peut merger des que l'ami @gustavi aura repondu a ca alors :

Rebase sans conflit comme j'ai fait hier ? Du coup je peux merger quand l'ami Travis sera content ?

@gustavi

This comment has been minimized.

Copy link
Member Author

commented Feb 4, 2015

Un conflit sur du docstring et un alignement dans le models.py. Rien de grave.

Eskimon added a commit that referenced this pull request Feb 4, 2015

@Eskimon Eskimon merged commit b1036d5 into zestedesavoir:dev Feb 4, 2015

1 check failed

continuous-integration/travis-ci The Travis CI build failed
Details

@Eskimon Eskimon added this to the Version 1.6 milestone Feb 4, 2015

@Eskimon Eskimon added S-BUG C-Back and removed S-BUG labels Feb 4, 2015

@gustavi gustavi deleted the gustavi:clean_mp_b branch Feb 10, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.