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

Met à jour les dépendances du backend #4657

Merged
merged 20 commits into from
Sep 23, 2017
Merged

Conversation

SpaceFox
Copy link
Contributor

@SpaceFox SpaceFox commented Sep 22, 2017

Q R
Type de modification MàJ des dépendances
Ticket(s) (issue(s)) concerné(s) #3898

Comme d'hab, un commit par dépendance pour pouvoir dégager facilement une éventuelle MàJ problématique, ou pour pouvoir tester un état précis.

Dans l'ordre des commits :

  1. MàJ de sécurité
  2. MàJ mineures
  3. MàJ majeures

En supposant que les auteur respectent un minimum la cohérence des numéros de version…

QA

  • Vérifier que rien n'est cassé, en particulier au niveau des fonctionnalités Git

@coveralls
Copy link

Coverage Status

Coverage decreased (-32.2%) to 57.232% when pulling d57b87a on SpaceFox:update_deps into e3ea93a on zestedesavoir:dev.

@artragis
Copy link
Member

y'a un test qui a échoué à cause de la maj de gitpython. Faut voir ce qui a changé.

L'autre fail vient de flake8-quotes.

@motet-a
Copy link
Contributor

motet-a commented Sep 23, 2017

Mergez d’abord #4658. (Fait)

@SpaceFox
Copy link
Contributor Author

Ce qui a changé, c'est la gestion des sha1 invalides quand on veut afficher un diff : visiblement le type d'erreur a changé. Je corrige ça.

@SpaceFox
Copy link
Contributor Author

Et voilà, ça devrait rouler.

@coveralls
Copy link

coveralls commented Sep 23, 2017

Coverage Status

Coverage remained the same at 89.476% when pulling f5bda7c on SpaceFox:update_deps into f20d427 on zestedesavoir:dev.

@artragis
Copy link
Member

Vue la quantité de test, je propose qu'on merge.

@motet-a
Copy link
Contributor

motet-a commented Sep 23, 2017

Corrige #3898.

@motet-a motet-a changed the title Update deps Met à jour les dépendances du backend Sep 23, 2017
@motet-a motet-a added the C-Back Concerne le back-end Django label Sep 23, 2017
@gllmc
Copy link
Member

gllmc commented Sep 23, 2017

J'ai fait un test rapide des fonctions principales du site avec les dépendances mises à jour et tout fonctionne, donc je merge. Merci @SpaceFox !

@gllmc gllmc added this to the Version de développement milestone Sep 23, 2017
@gllmc gllmc merged commit 5686c7c into zestedesavoir:dev Sep 23, 2017
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

5 participants