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

chore: Un peu d'amour dans le backend (dépendences, simplifications de syntaxe) #1435

Merged
merged 7 commits into from
Jan 18, 2023

Conversation

HEYGUL
Copy link
Contributor

@HEYGUL HEYGUL commented Jan 17, 2023

❤️ Contexte

  • Vous aimez le backend ?
  • Oui !
  • J'ai rien entendu ! Vous ai-mez le back-end ?
  • Ouiiiiiiii !

Nous aussi ! Alors voilà un peu d'amour ❤️ !

Avec cette PR, on en finit avec :

  • str(une variable de type UUID) dans un json ➡️ avec le monkeypatch présent dans api/__init__.py, on peut utiliser la variable directement.
  • f"{une variable de type str} ➡️ autant utiliser directement la variable.
  • le warning sur la run loop lors des tests ➡️ on monte la version de pytest-asyncio.
  • les dépendences de dev qui ne sont pas dans le groupe dev ➡️ on les déplace au bon endroit dans le pyproject.toml. On en profite pour utiliser le bon groupe avec poetry install #package# --group dev.

On en profite pour monter de version poetry vers la 1.3.2.

🏝️ Comment tester

Les tests python passent ? Alors tout va bien 🤗

@HEYGUL HEYGUL self-assigned this Jan 17, 2023
@HEYGUL HEYGUL force-pushed the chore/some-love-to-the-backend branch from 9402048 to dc1a14c Compare January 17, 2023 03:00
Copy link
Contributor

@lionelB lionelB left a comment

Choose a reason for hiding this comment

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

le buildpack qu'on utilise a pour valeur par défaut la 1.2.2 (à moins qu'on fournisse une variable d'environnement POETRY_VERSION. Il faudra penser à mettre à jour le buildpack ou les variable de template app

@HEYGUL HEYGUL force-pushed the chore/some-love-to-the-backend branch from dc1a14c to f0bdcd6 Compare January 17, 2023 12:27
@jonathanperret
Copy link
Contributor

le buildpack qu'on utilise a pour valeur par défaut la 1.2.2 (à moins qu'on fournisse une variable d'environnement POETRY_VERSION. Il faudra penser à mettre à jour le buildpack ou les variable de template app

Tout à fait ! On peut s’en sortir avec la variable d’environnement mais dans l’idéal je me dis qu’on pourrait modifier le buildpack pour accepter la version dans un fichier, pour éviter d’avoir à se souvenir de maintenir la variable sur tous les environnements.

@HEYGUL HEYGUL requested a review from vjousse January 17, 2023 16:56
@HEYGUL HEYGUL force-pushed the chore/some-love-to-the-backend branch from f0bdcd6 to f8cab04 Compare January 17, 2023 21:14
@HEYGUL HEYGUL force-pushed the chore/some-love-to-the-backend branch from f8cab04 to 22ae307 Compare January 17, 2023 21:19
@HEYGUL
Copy link
Contributor Author

HEYGUL commented Jan 17, 2023

le buildpack qu'on utilise a pour valeur par défaut la 1.2.2 (à moins qu'on fournisse une variable d'environnement POETRY_VERSION. Il faudra penser à mettre à jour le buildpack ou les variable de template app

Tout à fait ! On peut s’en sortir avec la variable d’environnement mais dans l’idéal je me dis qu’on pourrait modifier le buildpack pour accepter la version dans un fichier, pour éviter d’avoir à se souvenir de maintenir la variable sur tous les environnements.

J'ai fait un fork et ouvert une PR, cela semble fonctionner sur scalingo.
jonathanperret/scalingo-python-poetry-buildpack#1

@HEYGUL HEYGUL force-pushed the chore/some-love-to-the-backend branch from dba796b to 79b6785 Compare January 18, 2023 03:10
@lionelB lionelB merged commit de0a354 into master Jan 18, 2023
@lionelB lionelB deleted the chore/some-love-to-the-backend branch January 18, 2023 09:12
@service-dev-gip-inclusion
Copy link
Collaborator

🎉 This PR is included in version 1.185.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants