Skip to content

[DRAFT] Suite suppression "compile time env"#5556

Draft
thbar wants to merge 3 commits into
masterfrom
debt/more-compile-env-removal
Draft

[DRAFT] Suite suppression "compile time env"#5556
thbar wants to merge 3 commits into
masterfrom
debt/more-compile-env-removal

Conversation

@thbar

@thbar thbar commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Dans cette PR, je continue le travail suivant:

Je remplace prod.exs (qui embarque des variables d'ENV à la compilation) par un mécanisme de génération de configuration.

Un module est compilé, qui prend en entrée une map et l'atome de l'environnement (ici :prod). Par conséquent, les tests peuvent vérifier que la configuration de prod est correcte.

Le runtime se contente d'appeler 1 méthode de ce module (et là, si des variables manquent, au cas par cas c'est une exception qui est levée).

J'ouvre pour qu'on puisse regarder un peu, toutefois vu la surface, je testerai sur prochainement et vérifierai que la config est identique via SSH, avant/après.

Etapes

J'ai commencé par ré-activer le check credo correspondant sur prod.exs, qui a montré les warnings suivants:

┃ 
┃ [F] → get_env forbidden at compile time; move to config/runtime.exs
Warning: ┃       config/prod.exs:32:2 #()
┃ [F] → get_env forbidden at compile time; move to config/runtime.exs
Warning: ┃       config/prod.exs:29:2 #()
┃ [F] → get_env forbidden at compile time; move to config/runtime.exs
Warning: ┃       config/prod.exs:25:2 #()
┃ [F] → get_env forbidden at compile time; move to config/runtime.exs
Warning: ┃       config/prod.exs:24:2 #()
┃ [F] → get_env forbidden at compile time; move to config/runtime.exs
Warning: ┃       config/prod.exs:20:2 #()
┃ [F] → get_env forbidden at compile time; move to config/runtime.exs
Warning: ┃       config/prod.exs:19:2 #()
┃ [F] → get_env forbidden at compile time; move to config/runtime.exs
Warning: ┃       config/prod.exs:7:2 #()

Puis j'ai corrigé.

@thbar thbar added the dette technique Entretien & maintenance générale, nécessaire pour que le code reste de bonne qualité label Jul 3, 2026
@thbar

thbar commented Jul 6, 2026

Copy link
Copy Markdown
Contributor Author

@etalab/transport-tech je déploie sur prochainement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dette technique Entretien & maintenance générale, nécessaire pour que le code reste de bonne qualité

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant