Refactor the application package within ruff's complexity limits
Maintainer antworten meist innerhalb von 2 Tagen
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- Über eine Woche
- Anfängerfreundlichkeit
- 18/100
Rechercherichtung
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.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- Python
- Sterne
- 18
- Forks
- 33
- Ø Merge
- 1 T. 18 Std.
- Gemergte PRs (30 T.)
- 23
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus PolicyEngine/policyengine-api
-
Schwierigkeit 1/5 1-3 Stunden Anfängerfreundlichkeit 88/100
PolicyEngine/policyengine-api#3911 ·
Maintainer antworten meist innerhalb von 2 Tagen
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 82/100
PolicyEngine/policyengine-api#3910 ·
Maintainer antworten meist innerhalb von 2 Tagen
-
Register weighted-microdata rules: weights aren't valuesEvtl. vergeben @anth-volk hat das heute übernommen. Offen
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 18/100
PolicyEngine/policyengine-api#3937 ·
Maintainer antworten meist innerhalb von 2 Tagen
-
Remove the unused economy comparison moduleEvtl. vergeben @anth-volk hat das heute übernommen. Offen
Schwierigkeit 3/5 Ein halber Tag Anfängerfreundlichkeit 18/100
PolicyEngine/policyengine-api#3935 ·
Maintainer antworten meist innerhalb von 2 Tagen
-
Refactor tests, scripts and the rules tool, then select every ruff rule (ALL)Evtl. vergeben @anth-volk hat das heute übernommen. Offen
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 15/100
PolicyEngine/policyengine-api#3931 ·
Maintainer antworten meist innerhalb von 2 Tagen
Alle Issues in PolicyEngine/policyengine-api
Ähnliche Issues
-
fix formatting of the sample database page of the docsEvtl. vergeben @Aaqibhafeezkhan hat das heute übernommen. Offendocumentation good first issue help wanted
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 72/100
chaoss/CollectOSS#544 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
area/plugin good first issue
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 85/100
kestra-io/plugin-soda#137 ·
-
Schwierigkeit 2/5 Ein halber Tag Anfängerfreundlichkeit 62/100
ibm-granite/granite.build#504 ·
Maintainer antworten meist innerhalb von 2 Tagen
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
Maintainer antworten meist innerhalb von 1 Tag