Remove redundant ways to load environment variables
Les mainteneurs répondent en général sous 10 jours
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 45/100
Piste de recherche
Commencez par comparer backend/app/core/config.py, autour de la définition des settings, avec backend/app/load_env.py, puis recherchez dans le dépôt les références à load_env.py. Supprimez le loader redondant et le code qui l’appelle, tout en conservant le chemin des settings de config.py. C’est terminé lorsqu’il ne reste plus aucune référence à load_env.py et que les variables d’environnement utilisent l’unique approche prévue.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Which feature or component needs enhancement?
The way we manage environment variables in the project
Describe the current behavior
We seem to have two redundant approaches to loading environment variables from the *.env files.
https://github.com/ProjectTech4DevAI/kaapi-guardrails/blob/1d62fc24821ca6e7b5c7d8aeb473fc9052787a4d/backend/app/core/config.py#L94
and
https://github.com/ProjectTech4DevAI/kaapi-guardrails/blob/1d62fc24821ca6e7b5c7d8aeb473fc9052787a4d/backend/app/load_env.py#L5
Describe the enhancement you'd like
The settings variable exposed in config.py seems to be the one we are relying heavily on throughout the codebase. So we can remove the load_env.py and all calling code referencing that file.
Why is this enhancement needed?
This will prevent confusing overrides of environment variables due to the two approaches interacting in unexpected manner.
- Langage dominant
- Python
- Étoiles
- 5
- Forks
- 3
- Merge moyen
- 9 j 13 h
- PR mergées (30 j)
- 2
Préparer son environnement
Nous n'avons pas encore vérifié les fichiers d'installation de ce projet. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de ProjectTech4DevAI/kaapi-guardrails
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
ProjectTech4DevAI/kaapi-guardrails#94 ·
Les mainteneurs répondent en général sous 10 jours
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 65/100
ProjectTech4DevAI/kaapi-guardrails#23 ·
Les mainteneurs répondent en général sous 10 jours
-
Callback Enhancement: Include intermediate responsesPeut-être pris @Prajna1999 l’a pris il y a 21 jours. Ouverte
ProjectTech4DevAI/kaapi-guardrails#158 · 1 personne assignée ·
Les mainteneurs répondent en général sous 10 jours
-
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
ProjectTech4DevAI/kaapi-guardrails#144 ·
Les mainteneurs répondent en général sous 10 jours
-
Kaapi Console: Display config idOuverte
Difficulté 3/5 1-2 jours Accessibilité débutants 55/100
ProjectTech4DevAI/kaapi-guardrails#137 ·
Les mainteneurs répondent en général sous 10 jours
Toutes les issues de ProjectTech4DevAI/kaapi-guardrails
Issues similaires
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 72/100
letsencrypt/cp-cps#353 ·
-
Marble Madness II is missingOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
PedestrianDynamics/pyFDS-Evac#394 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
DOI-USGS/pywatershed#421 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
python-pillow/Pillow#10087 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour