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

feat: utilisation du score de qualité de DI #193

Merged
merged 10 commits into from
Jan 22, 2025
Merged

Conversation

ggounot
Copy link
Contributor

@ggounot ggounot commented Jan 21, 2025

Un score de qualité minimum peut être spécifié via la variable d'environnement DATA_INCLUSION_SCORE_QUALITE_MINIMUM.

Lorsqu'il est spécifié, il est utilisé en paramètre de recherche de services DI.

Implémentation du score du qualité dans les outils de test FakeDataInclusionClient et make_service_data().

Ajout d'un test de recherche DI avec score de qualité minimum.

@ggounot ggounot force-pushed the feat/score-qualite branch from 7d6cabd to 60ad5a3 Compare January 21, 2025 19:52
@ggounot ggounot requested a review from jbuget January 21, 2025 19:52
Copy link
Contributor

@jbuget jbuget left a comment

Choose a reason for hiding this comment

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

nitpick: ajouter la variable dans le fichier envs-example/dev.env

a part ça, le code est propre et fonctionne. merci pour les TU

@ggounot
Copy link
Contributor Author

ggounot commented Jan 22, 2025

nitpick: ajouter la variable dans le fichier envs-example/dev.env

a part ça, le code est propre et fonctionne. merci pour les TU

Je ne l'avais pas fait car ça initialisait la variable avec une chaîne vide. J'ai changé la gestion du type pour pouvoir le faire : 617d525

@ggounot ggounot requested a review from jbuget January 22, 2025 08:28
@ggounot ggounot force-pushed the feat/score-qualite branch from 4b430e1 to 617d525 Compare January 22, 2025 08:29
@ggounot ggounot enabled auto-merge (squash) January 22, 2025 08:43
Copy link
Contributor

@jbuget jbuget left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@ggounot ggounot merged commit 888b114 into main Jan 22, 2025
7 checks passed
@ggounot ggounot deleted the feat/score-qualite branch January 22, 2025 10:09
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.

2 participants