Remove redundant ways to load environment variables
Maintainer thường phản hồi trong vòng 10 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 45/100
Hướng nghiên cứu
Bắt đầu bằng cách so sánh backend/app/core/config.py, ở phần định nghĩa settings, với backend/app/load_env.py, sau đó tìm kiếm trong repository các tham chiếu đến load_env.py. Xóa loader dư thừa và mã gọi nó, đồng thời giữ lại đường dẫn settings của config.py. Hoàn tất khi không còn tham chiếu nào đến load_env.py và các biến môi trường sử dụng cách tiếp cận duy nhất đã được định sẵn.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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.
- Ngôn ngữ chính
- Python
- Star
- 5
- Fork
- 3
- Merge trung bình
- 9 ngày 13 giờ
- Pull request đã merge (30 ngày)
- 2
Chuẩn bị môi trường
Chúng tôi chưa kiểm tra các tệp thiết lập môi trường của dự án này. Hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của ProjectTech4DevAI/kaapi-guardrails
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
ProjectTech4DevAI/kaapi-guardrails#94 ·
Maintainer thường phản hồi trong vòng 10 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 65/100
ProjectTech4DevAI/kaapi-guardrails#23 ·
Maintainer thường phản hồi trong vòng 10 ngày
-
Callback Enhancement: Include intermediate responsesCó thể đã có người làm @Prajna1999 đã nhận 21 ngày trước. Đang mở
ProjectTech4DevAI/kaapi-guardrails#158 · 1 người được giao ·
Maintainer thường phản hồi trong vòng 10 ngày
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100
ProjectTech4DevAI/kaapi-guardrails#144 ·
Maintainer thường phản hồi trong vòng 10 ngày
-
Kaapi Console: Display config idĐang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 55/100
ProjectTech4DevAI/kaapi-guardrails#137 ·
Maintainer thường phản hồi trong vòng 10 ngày
Tất cả issue của ProjectTech4DevAI/kaapi-guardrails
Issue tương tự
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 72/100
letsencrypt/cp-cps#353 ·
-
Marble Madness II is missingĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
PedestrianDynamics/pyFDS-Evac#394 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
DOI-USGS/pywatershed#421 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
python-pillow/Pillow#10087 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày