Refactor the application package within ruff's complexity limits
Maintainer thường phản hồi trong vòng 2 ngày
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 18/100
Hướng nghiên cứu
Start by running ruff over policyengine_api/ to list the C901, PLR and PLR2004 violations in the v2 services, Flask routes, data layer, migration logging, ASGI shell and v1 services. Pull request #3933 is already open against this work, so review it before touching any file. Done means ruff reports no violations in those paths, with characterization tests pinning the existing responses, queries, writes and log records.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Phase 1 step 1.8, group 8b, part 2 of 3: refactor the rest of policyengine_api/ (v2 services, Flask routes, the data layer, migration logging and flags, the ASGI shell and the v1 services) so every function is within ruff's complexity limits (C901 and the PLR limits), and name its magic numbers (PLR2004).
Behaviour must not change: the same responses, queries and writes in the same order, log records and published schemas. Flask views, FastAPI routes, Pydantic validators and SQLAlchemy listeners keep their external signatures. Pin behaviour with characterization tests before refactoring.
- Ngôn ngữ chính
- Python
- Star
- 18
- Fork
- 33
- Merge trung bình
- 1 ngày 18 giờ
- Pull request đã merge (30 ngày)
- 23
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
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 PolicyEngine/policyengine-api
-
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 88/100
PolicyEngine/policyengine-api#3911 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 82/100
PolicyEngine/policyengine-api#3910 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Register weighted-microdata rules: weights aren't valuesCó thể đã có người làm @anth-volk đã nhận hôm nay. Đang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 18/100
PolicyEngine/policyengine-api#3937 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Remove the unused economy comparison moduleCó thể đã có người làm @anth-volk đã nhận hôm nay. Đang mở
Độ khó 3/5 Nửa ngày Mức phù hợp với người mới 18/100
PolicyEngine/policyengine-api#3935 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Refactor tests, scripts and the rules tool, then select every ruff rule (ALL)Có thể đã có người làm @anth-volk đã nhận 1 ngày trước. Đang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 15/100
PolicyEngine/policyengine-api#3931 ·
Maintainer thường phản hồi trong vòng 2 ngày
Tất cả issue của PolicyEngine/policyengine-api
Issue tương tự
-
`burr` fails to start when the Python path has a space in itCó thể đã có người làm @enekoreto đã nhận hôm nay. Đang mởstatus/needs-triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 5 ngày
-
gemini 出现了新的域名需要加入Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 72/100
-
Độ khó 2/5 Nửa ngày Mức phù hợp với người mới 72/100
debpalash/VoiceStudio#2903 ·
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 72/100
dagster-io/community-integrations#367 ·
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 74/100
aws-samples/sample-per-model-bedrock#243 ·
Maintainer thường phản hồi trong vòng 1 ngày