Refactor within ruff's complexity limits, then select every ruff rule (ALL)
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 15/100
Línea de trabajo
The issue names no files. Start by running ruff with the complexity rules (C901 and the PLR limits) enabled to list the functions over the limits, and check the ruff config in the repository for the current selection. Before touching any function, add characterization tests that pin its outputs, errors, logs, and OpenAPI output. Done means the listed functions are under the limits, behaviour is unchanged, and ruff passes with select = ["ALL"] and only the stated global ignores.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Phase 1 step 1.8, group 8b, the last group: refactor every function over ruff's complexity limits (C901, and the PLR limits on arguments, branches, statements and returns), name the magic numbers (PLR2004), then switch ruff to select = ["ALL"].
Behaviour must not change. That covers the same outputs, errors, logs, Modal function signatures, library APIs other projects import, and byte-identical OpenAPI documents. Pin behaviour with characterization tests before each refactor.
The only global ignores are the two rules that conflict with ruff format (COM812, ISC001) and one rule of each mutually exclusive pydocstyle pair (D203, D213).
- Lenguaje dominante
- Python
- Estrellas
- 1
- Forks
- 1
- Merge medio
- 1 d 10 h
- PR fusionados (30 d)
- 53
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-sim-api
-
Integration tests fail due to missing UV virtual environment setupPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
PolicyEngine/policyengine-sim-api#314 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 4/5 3-5 días Aptitud para principiantes 22/100
PolicyEngine/policyengine-sim-api#814 ·
Los mantenedores suelen responder en 1 día
-
Register weighted-microdata rules: weights aren't valuesPosiblemente ocupada @anth-volk la tomó hace 2 días. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 30/100
PolicyEngine/policyengine-sim-api#806 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 25/100
PolicyEngine/policyengine-sim-api#805 ·
Los mantenedores suelen responder en 1 día
-
Enable ruff argument, idiom and pytest-style rules (ARG, B, C4, FBT, PERF, PLW, PT, PTH, RUF, SIM and others)Posiblemente ocupada @anth-volk la tomó hace 2 días. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 22/100
PolicyEngine/policyengine-sim-api#801 ·
Los mantenedores suelen responder en 1 día
Todos los issues de PolicyEngine/policyengine-sim-api
Issues similares
-
docs: the dev-setup table and AGENTS.md name the reviewed_by guard that CI's Changelog job runsAbiertoarea: tools documentation good first issue ready for a contributor
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
eyalgolan/statecraft-israel#873 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Add cover art to song rowsAbiertoenhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 5 días
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día