Skip to content

GitLab

Closed
Created by Jonas SPRENGER@JonasSprengerDeveloper
Report abuse

Resolve "durs-core: load conf from environment variables as a priority"

Ticket: #141 (closed)

Première MR minimal répondant au ticket donc:

  • Charge une config optionnel/partielle par variable d’environnement qui est prioritaire par rapport à la config chargé depuis le fichier de conf.

Par contre la lecture et écriture du fichier de conf n'a pas évolué.

  • Il n'est donc pas encore optionnel.
  • Le trait Merge n'est donc pas utilisé.

Je compte tenter un refacto avant de créer une version DursUserConfV3 qui rendras les configurations depuis le fichier optionnelles.

Note sur le dev:

  • Pour pouvoir deserializer l'enum ResourceUsage depuis les variables d'env il faut ajouter la macro #[serde(field_identifier)], par contre cela empêche la résolution de la macro #[derive(Serialize)]. La solution que j'ai trouvé est d'implémenter le trait Serialize sans passer par la macro, donc: impl Serialize for ResourceUsage.
Edited by Jonas SPRENGER