Enable ruff security, logging and debugging rules (S, T20, LOG, G, DTZ, ERA, TD, FIX, EXE)
Maintainer antworten meist innerhalb von 2 Tagen
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 18/100
Rechercherichtung
The payload names no specific files, only the ruff rule groups (S, T20, LOG, G, DTZ, ERA, TD, FIX, EXE) and the S101 asserts, T20 prints, and DTZ datetimes they touch. Start with the ruff configuration in the repository and linked pull request #3915 to see which rules are already enabled and what has been fixed. Done means the new rules pass with each finding either fixed or suppressed with a stated reason, and error responses and published schemas are unchanged.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Phase 1 step 1.8, group 6 of the rules workstream: enable ruff's security, logging and debugging rules (S, T20, LOG, G, DTZ, ERA, TD, FIX, EXE).
- Security (
S): fix each finding, or keep it with an inline reason where it's a false positive (fake secrets in tests, repository commands run by tests, fixed container paths). - Asserts outside tests (
S101): replace type-narrowing asserts with explicit checks, without changing any error response. - Prints (
T20): remove debug prints from application code; command-line scripts keep their output, with a reason. - Logging (
LOG,G): module loggers and lazy formatting, with identical message text. - Datetimes (
DTZ): never switch a value between naive and aware; keep naive values that must stay naive, with a reason. - Leftovers (
ERA,TD,FIX,EXE): delete commented-out code, turn TODOs into plain notes or issue links, and match shebangs with the executable bit.
Error responses and published schemas must not change.
- Vorherrschende Sprache
- Python
- Sterne
- 18
- Forks
- 33
- Ø Merge
- 1 T. 20 Std.
- Gemergte PRs (30 T.)
- 27
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
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 22/100
PolicyEngine/policyengine-api#3946 ·
Maintainer antworten meist innerhalb von 2 Tagen
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 22/100
PolicyEngine/policyengine-api#3945 ·
Maintainer antworten meist innerhalb von 2 Tagen
-
Register weighted-microdata rules: weights aren't valuesEvtl. vergeben @anth-volk hat das vor 2 Tagen übernommen. Offen
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 18/100
PolicyEngine/policyengine-api#3937 ·
Maintainer antworten meist innerhalb von 2 Tagen
Alle Issues in PolicyEngine/policyengine-api
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
noontide-co/mainbranch#1211 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
[1.x Bug] run-operation --sql reports success but rolls back writes on transaction-capable adaptersOffentriage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 64/100
dbt-labs/dbt#16784 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug needs-triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
Maintainer antworten meist innerhalb von 1 Tag
-
bug settings ui
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
DewaldOosthuizen/py-tray-command-launcher#162 ·
Maintainer antworten meist innerhalb von 1 Tag
-
run_all sleeps after the final job despite documenting delays between jobsEvtl. vergeben @Gambit-Checkmate hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100