Enable ruff argument, idiom and pytest-style rules (ARG, B, C4, FBT, PERF, PLW, PT, PTH, RUF, SIM and others)
Maintainer antworten meist innerhalb von 1 Tag
Ein zugehöriger Pull Request wurde bereits gemerged.
- #802 von @anth-volk — gemerged
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 22/100
Rechercherichtung
This is a maintainer-led lint rollout across the whole repository, so it is not a good first pick. Start with the ruff rule selection in the project's lint configuration, then run ruff on one rule group at a time to see the violations. Done means each enabled group is clean, with deliberate exceptions noted, and the published schemas and HTTP API are unchanged.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Phase 1 step 1.8, group 8a of the rules workstream: enable ruff's argument, idiom and pytest-style rules (ARG, ASYNC, B, C4, FBT, PD, PERF, PGH, PLW, PT, PTH, RUF, SIM). Group 8b follows with the complexity refactors (C901, PLR) and the switch to select = ["ALL"].
- Unused arguments (
ARG): remove, underscore or explain each. Request side-effect fixtures withusefixtures. Never rename a parameter that callers pass by keyword. - Boolean arguments (
FBT): make them keyword-only where every caller is in the repository. - Idioms (
C4,SIM,PERF,RUF,PTH,PLW,B): apply equivalent rewrites; keep deliberate exceptions with a reason. - Tests (
PT): pytest idioms, withpytest.raises(match=...)on the expected message. The shared rules tool's tests use unittest on purpose and are exempt fromPT.
Published schemas, OpenAPI documents and the HTTP API must not change.
- Vorherrschende Sprache
- Python
- Sterne
- 1
- Forks
- 1
- Ø Merge
- 1 T. 19 Std.
- Gemergte PRs (30 T.)
- 77
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-sim-api
-
Accept the withheld-area code lists on UK local-area breakdown results before any executor sends themEvtl. vergeben @anth-volk hat das vor 1 Tag übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 67/100
PolicyEngine/policyengine-sim-api#977 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Update policyengine to 6.11.1Evtl. vergeben @policyengine hat das vor 2 Tagen übernommen. Offen
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 68/100
PolicyEngine/policyengine-sim-api#916 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Update policyengine to 6.7.3Evtl. wieder frei Ein Pull Request dazu wurde ohne Merge geschlossen. Offen
Schwierigkeit 2/5 Ein halber Tag Anfängerfreundlichkeit 70/100
PolicyEngine/policyengine-sim-api#823 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Integration tests fail due to missing UV virtual environment setupEvtl. vergeben Ein verknüpfter Pull Request ist offen oder bereits gemergt. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
PolicyEngine/policyengine-sim-api#314 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Move the changelog to Towncrier fragments and build CHANGELOG.md at each deployEvtl. vergeben @anth-volk hat das heute übernommen. Offen
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100
PolicyEngine/policyengine-sim-api#978 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in PolicyEngine/policyengine-sim-api
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
microsoft/agent-lightning#661 ·
Maintainer antworten meist innerhalb von 2 Tagen
-
area/profiles comp/cli P3 type/feature
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
NousResearch/hermes-agent#146372 ·
Maintainer antworten meist innerhalb von 1 Tag
-
[BUG] OpenAI-compatible embedding model alias breaks embedder constructionEvtl. vergeben Ein verknüpfter Pull Request ist offen oder bereits gemergt. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
Maintainer antworten meist innerhalb von 1 Tag
-
[oblt-aw][autodoc] Document obs-aw-automerge contract and fix automerge doc inconsistenciesEvtl. vergeben @github-actions hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
Maintainer antworten meist innerhalb von 1 Tag
-
GeoSeries scale and skew tests fail with pytest 9.2.0Evtl. vergeben @jiayuasu hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
Maintainer antworten meist innerhalb von 1 Tag