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

Ajout d'un fichier de migration manquant #2527

Merged
merged 1 commit into from
Apr 13, 2015

Conversation

pierre-24
Copy link
Member

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

En testant la nouvelle commande de migration de Django 1.7 (python manage.py makemigrations) sur upstream/dev, je me suis rendu compte qu'il y avait le module forum qui n'avait pas été migré à un moment. Pour éviter les problèmes, le voici.

Note de QA

... Y'en a pas ? Je pense que si Travis est content (et c'est pas gagné, par les temps qui courent), ben c'est bon. Si vous y tenez absolument, un python manage.py migrate et Django est heureux :)

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 4bc807f on pierre-24:forgoten_forum_migration into fcba6f1 on zestedesavoir:dev.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 79.3% when pulling 4bc807f on pierre-24:forgoten_forum_migration into fcba6f1 on zestedesavoir:dev.

@pierre-24
Copy link
Member Author

Bon, Travis semble de bonne humeur, aujourd'hui.

@Eskimon
Copy link
Contributor

Eskimon commented Apr 11, 2015

C'est lié à cette PR je pense : #2457

@firm1 tu confirmes et donne un "GO pour merge" ?

@Eskimon
Copy link
Contributor

Eskimon commented Apr 13, 2015

reping @firm1 pour confirmation :)

@firm1
Copy link
Contributor

firm1 commented Apr 13, 2015

Je donne le GO pour merge.

Par contre, je ne sais pas s'il faut le faire dans cette PR ou crée une issue, mais ça serait cool que test apparaisse pour ce genre de situation pour éviter de se retrouver dans ce cas. Le test consisterait à demander à travis de lancer la commande de création des fichiers de migrations, et s'assurer qu'aucun fichier n'a été crée en plus.

Mais après un débat de 5 sec avec moi même, je pense qu'on devrait en faire une issue à part.

@SpaceFox
Copy link
Contributor

C'est clairement une issue à part oui, donc je merge.

SpaceFox added a commit that referenced this pull request Apr 13, 2015
Ajout d'un fichier de migration manquant
@SpaceFox SpaceFox merged commit c598117 into zestedesavoir:dev Apr 13, 2015
@SpaceFox SpaceFox added S-BUG Corrige un problème C-Back Concerne le back-end Django labels Apr 13, 2015
@SpaceFox SpaceFox added this to the Version 1.8 milestone Apr 13, 2015
@pierre-24 pierre-24 deleted the forgoten_forum_migration branch April 16, 2015 02:13
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 S-BUG Corrige un problème
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants