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 du code #1466

Merged
merged 10 commits into from
Sep 8, 2014
Merged

Nettoyage du code #1466

merged 10 commits into from
Sep 8, 2014

Conversation

firm1
Copy link
Contributor

@firm1 firm1 commented Sep 7, 2014

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

Cette PR vise principalement le nettoyage de fonctions devenues inutiles dans le code.

Note pour QA

Si les tests Travis passent avec succès, je pense que ça devrait être bon.

@pierre-24
Copy link
Member

Elle plante, mais c'est triché, elle plante sur npm --"

@Alex-D
Copy link
Contributor

Alex-D commented Sep 7, 2014

Je l'ai relancé.

@firm1
Copy link
Contributor Author

firm1 commented Sep 7, 2014

Visiblement NPM est cassé aujourd'hui.

@pierre-24
Copy link
Member

Oui. ping @Alex-D : une idée de pourquoi ? (t'as plus l'habitude de npm, je pense)

@Eskimon
Copy link
Contributor

Eskimon commented Sep 7, 2014

Par contre dire ca :

Si les tests Travis passent avec succès, je pense que ça devrait être bon

C'est un peu visé gros vu qu'on a pas une couverture de 100% ;)

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling b2c417b on firm1:fix-clean-code into * on zestedesavoir:dev*.

@Alex-D Alex-D added C-Back Concerne le back-end Django Facile Bon ticket pour débuter pour rejoindre le développement ! labels Sep 8, 2014
@Alex-D
Copy link
Contributor

Alex-D commented Sep 8, 2014

Les tests passent \o/

@pierre-24
Copy link
Member

Je tenterai de faire une QA exhaustive ce soir. Si c'est pas fait pour 22h, pingez moi :)

@pierre-24
Copy link
Member

TL;DR : à merger

Bon. J'ai fait le tour du site, mais c'est dur d'être exhaustif sur tout. À fortiori, j'ai rien trouvé comme bug, on peut donc considérer que ça marche et que cette PR est bonne. Ceci-dit, faudra être vigilant, parce que même avec ça et une couverture de 81%, on est pas à l'abri de surprises.

firm1 pushed a commit that referenced this pull request Sep 8, 2014
@firm1 firm1 merged commit 8868598 into zestedesavoir:dev Sep 8, 2014
@SpaceFox SpaceFox added this to the Version 1.1 milestone Sep 24, 2014
@firm1 firm1 deleted the fix-clean-code branch June 24, 2015 09:06
@firm1 firm1 restored the fix-clean-code branch March 2, 2018 17:35
@firm1 firm1 deleted the fix-clean-code branch November 5, 2019 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django Facile Bon ticket pour débuter pour rejoindre le développement !
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants