Skip to content
Snippets Groups Projects

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

Merge request reports

Checking pipeline status.

Closed by ÉloïsÉloïs 5 years ago (Jan 26, 2020 9:05pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading