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

Mise à jour triviale des dépendances Python #6340

Merged
merged 1 commit into from
Jun 26, 2022

Conversation

Situphen
Copy link
Member

Mise à jour triviale des dépendances Python

Je n'ai pas mis à jour django-oauth-toolkit car la nouvelle version contient des breaking change conséquents.

QA :

  • source zdsenv/bin/activate && make update && make zmd-start && make run-back
  • Vérifier que le site fonctionne correctement dans sa globalité
  • make generate-doc puis ouvrir doc/build/html/index.html dans un navigateur
  • S'assurer que la documentation n'est pas cassée

@Situphen Situphen added the C-Back Concerne le back-end Django label Jun 25, 2022
@philippemilink
Copy link
Member

Les tests ne passent pas à cause de changements cassants dans l'API Selenium (Deprecated find_element_by_* and find_elements_by_* are now removed).

@Situphen Situphen force-pushed the python-deps branch 4 times, most recently from 05f2b38 to 675c805 Compare June 25, 2022 15:49
@coveralls
Copy link

Coverage Status

Coverage remained the same at 87.99% when pulling 3cd99c3 on Situphen:python-deps into f171924 on zestedesavoir:dev.

@Situphen
Copy link
Member Author

C'est corrigé !

Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA OK ✔️

@philippemilink philippemilink merged commit d166c8c into zestedesavoir:dev Jun 26, 2022
@Situphen Situphen deleted the python-deps branch June 26, 2022 17:02
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
Archived in project
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants