Refactor the application package within ruff's complexity limits
Los mantenedores suelen responder en 2 días
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 18/100
Línea de trabajo
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.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Python
- Estrellas
- 18
- Forks
- 33
- Merge medio
- 1 d 19 h
- PR fusionados (30 d)
- 26
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de PolicyEngine/policyengine-api
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
PolicyEngine/policyengine-api#3911 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
PolicyEngine/policyengine-api#3910 ·
Los mantenedores suelen responder en 2 días
-
Register weighted-microdata rules: weights aren't valuesPosiblemente ocupada @anth-volk la tomó hace 1 día. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 18/100
PolicyEngine/policyengine-api#3937 ·
Los mantenedores suelen responder en 2 días
-
Remove the unused economy comparison modulePosiblemente ocupada @anth-volk la tomó hace 1 día. Abierto
Dificultad 3/5 Medio día Aptitud para principiantes 18/100
PolicyEngine/policyengine-api#3935 ·
Los mantenedores suelen responder en 2 días
-
Refactor tests, scripts and the rules tool, then select every ruff rule (ALL)Posiblemente ocupada @anth-volk la tomó hace 1 día. Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 15/100
PolicyEngine/policyengine-api#3931 ·
Los mantenedores suelen responder en 2 días
Todos los issues de PolicyEngine/policyengine-api
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
jbacalso24/mimry#291 ·
Los mantenedores suelen responder en 1 día
-
good first issue
Dificultad 2/5 Menos de una hora Aptitud para principiantes 82/100
eli-alvarado/us-watchlist-alerts#14 ·
Los mantenedores suelen responder en 1 día
-
from-review-extraction python severity:nit
Dificultad 2/5 1-3 horas Aptitud para principiantes 80/100
LearningCircuit/local-deep-research#7462 ·
Los mantenedores suelen responder en 2 días
-
mermaid_extract.py silently discards single-line state-diagram notes; sequence notes are keptAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
cathrynlavery/diagram-design#441 ·
Los mantenedores suelen responder en 2 días
-
[Cleanup] Include the public header in work_split.cppPosiblemente ocupada @blozano-tt la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
tenstorrent/tt-metal#60850 ·
Los mantenedores suelen responder en 1 día