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

Suppression d'un fichier inutile #2592

Merged
merged 2 commits into from
Apr 23, 2015
Merged

Conversation

firm1
Copy link
Contributor

@firm1 firm1 commented Apr 23, 2015

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

Cette PR supprime un fichier inutile dans le projet et rajoute un test pour vérifier que la commande de création des fixtures fonctionne bien.

Note pour QA

Normalement rien à faire à part lire le code et laisser Travis agir

@landscape-bot
Copy link

Code Health
Repository health decreased by 1% when pulling ddcee27 on firm1:clean_files into 6c05818 on zestedesavoir:dev.

@SpaceFox SpaceFox added Evolution C-Back Concerne le back-end Django labels Apr 23, 2015
@pierre-24
Copy link
Member

Question probablement stupide: rien à voir avec la commande load_factory_data mentionnée par la bientôt nouvelle doc ?

@firm1
Copy link
Contributor Author

firm1 commented Apr 23, 2015

Non ce fichier est orphelin et n'est utilisé nulle part. C'est bien pour ça
que j'ai rajouté le TU pour vérifier que la commande marche quand même sans
ce fichier.

Le jeu. 23 avr. 2015 20:16, Pierre Beaujean notifications@github.com a
écrit :

Question probablement stupide: rien à voir avec la commande
load_factory_data mentionnée par la bientôt nouvelle doc
https://github.com/zestedesavoir/zds-site/pull/2578/files#diff-a05aa1b9904eb47793ea72aca3b190b5R62
?


Reply to this email directly or view it on GitHub
#2592 (comment)
.

@pierre-24
Copy link
Member

Comme je disais, bête question. Je merge, then ;)

pierre-24 added a commit that referenced this pull request Apr 23, 2015
Suppression d'un fichier inutile
@pierre-24 pierre-24 merged commit a9c1b9b into zestedesavoir:dev Apr 23, 2015
@pierre-24
Copy link
Member

(faudrait pas un ticket pour le suivi ?)

@pierre-24 pierre-24 added this to the Version 1.9 milestone Apr 23, 2015
@firm1
Copy link
Contributor Author

firm1 commented Apr 23, 2015

J'ai pensé mais je savais pas quoi mettre dans le ticket. Si quelqu'un a
une idée ?

Le jeu. 23 avr. 2015 22:50, Pierre Beaujean notifications@github.com a
écrit :

(faudrait pas un ticket pour le suivi ?)


Reply to this email directly or view it on GitHub
#2592 (comment)
.

@pierre-24
Copy link
Member

En fait, on m'as donné les droits de merge mais on a oublié de me donner le mode d'emploi concernant milestones et le reste :p Je m'en vais de ce pas MP notre nouveau CDP !

@SpaceFox SpaceFox modified the milestones: Version 15.5.1, Version de développement May 26, 2015
@firm1 firm1 deleted the clean_files branch June 24, 2015 09:10
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants