Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

Refactor within ruff's complexity limits, then select every ruff rule (ALL)

Geschlossen
  • Stimmt etwas nicht?
#803 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Ein zugehöriger Pull Request wurde bereits gemerged.

  • #804 von @anth-volk — gemerged

Bewertung

Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Anfängerfreundlichkeit
15/100
Issue-Typ
Refactoring
Klarheit
Größtenteils klar
Aktivitätsstatus
Aktiv
Tech-Stack
python
Bereich
devtools

Rechercherichtung

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.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

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).

Vorherrschende Sprache
Python
Sterne
1
Forks
1
Ø Merge
1 T. 18 Std.
Gemergte PRs (30 T.)
59

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus PolicyEngine/policyengine-sim-api

Alle Issues in PolicyEngine/policyengine-sim-api

Ähnliche Issues

Weitere Issues zu Python

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.