Remove redundant ways to load environment variables
メンテナーはふだん 10 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
まず、settings の定義付近にある backend/app/core/config.py と backend/app/load_env.py を比較し、次にリポジトリ内で load_env.py への参照を検索します。冗長な loader と、それを呼び出しているコードを削除しつつ、config.py の settings パスは維持します。load_env.py への参照が残っておらず、環境変数が意図された単一のアプローチを使用していれば完了です。
索引モデルが issue の本文から書いたものです。
説明
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.
- 主要言語
- Python
- スター
- 5
- フォーク
- 3
- 平均マージ
- 9日 13時間
- マージ済み PR（30日）
- 2
環境構築
このプロジェクトの環境構築ファイルはまだ確認していません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
ProjectTech4DevAI/kaapi-guardrails のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
ProjectTech4DevAI/kaapi-guardrails#94 ·
メンテナーはふだん 10 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 65/100
ProjectTech4DevAI/kaapi-guardrails#23 ·
メンテナーはふだん 10 日以内に返信
-
Callback Enhancement: Include intermediate responses対応中かも @Prajna1999 が 21 日前に担当しました。 オープン
ProjectTech4DevAI/kaapi-guardrails#158 · 担当者 1 名 ·
メンテナーはふだん 10 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
ProjectTech4DevAI/kaapi-guardrails#144 ·
メンテナーはふだん 10 日以内に返信
-
難易度 3/5 1〜2日 初心者へのやさしさ 55/100
ProjectTech4DevAI/kaapi-guardrails#137 ·
メンテナーはふだん 10 日以内に返信
ProjectTech4DevAI/kaapi-guardrails の issue をすべて見る
似ている issue
-
bug server
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
sportsdataverse/sportsdataverse-py#641 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
googleapis/google-cloud-python#18532 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信