Enable ruff exception rules (TRY, EM, BLE)
Maintainer antworten meist innerhalb von 2 Tagen
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 38/100
Rechercherichtung
The issue names no specific files. Start by enabling the TRY, EM and BLE rule groups in the ruff lint configuration and listing the violations they report. Each fix needs a judgment call: narrowing a broad except, moving raises out of try blocks, or keeping a Pydantic validator's exception type unchanged. Done when ruff passes with these rules on and the existing tests pass, with message text and published schemas unchanged.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Phase 1 step 1.8, group 5 of the rules workstream: enable ruff's exception rules (TRY, EM, BLE).
- Messages: build each exception message before the
raise(EM, TRY003). - Raises inside
tryblocks: move them into helpers so a handler can't catch its own raise (TRY301, TRY300). - Broad catches: decide each
except Exception(BLE001). Narrow it where the possible errors are known; otherwise keep it with an inline reason. - Exception types: use
TypeErrorfor type checks (TRY004) unless a caller, a boundary or a test depends on the current type. Never change the type raised inside a Pydantic validator, whereTypeErrorturns a validation error into a 500.
Message text and published schemas must not change.
- 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 vor 1 Tag ü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 vor 1 Tag ü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 vor 1 Tag ü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
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
cathrynlavery/diagram-design#441 ·
Maintainer antworten meist innerhalb von 2 Tagen
-
[Cleanup] Include the public header in work_split.cppEvtl. vergeben @blozano-tt hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
tenstorrent/tt-metal#60850 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Maintainer antworten meist innerhalb von 1 Tag
-
backend::sglang bug language::python observability
Schwierigkeit 2/5 Ein halber Tag Anfängerfreundlichkeit 65/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 75/100
OpenShot/openshot-qt#6202 ·
Maintainer antworten meist innerhalb von 1 Tag