astral-sh/ruff

`N999` (`invalid-module-name`) mixes correctness and stylistic recommendations

Offen

#12.475 geöffnet am 23.07.2024

Auf GitHub ansehenContributor Guide
 (11 Kommentare) (2 Reaktionen) (0 zugewiesene Personen)Rust (2.088 Forks)batch import
help wantedrule

Repository-Metriken

Stars
 (47.527 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 3T 8h) (573 gemergte PRs in 30 T)

Beschreibung

Not only does rule N999 check that the PEP8 convention is respected, but also checks that the module name is syntactically correct. The latter is not part of PEP8. This is a problem for me currently working on "old non linted" code. I have to ignore this rule because of some modules whose names are syntactically correct but sometimes with capital letters. I think a E rule is missing that would only include the syntax checking of N999. (If I were you I would also remove that part from N999 for consistency)

Contributor Guide

Research-Richtung
Untersuche, wie Lint Regeln in Ruffs Codebasis definiert sind, finde die Implementierung der Regel N999, verstehe, wie sie PEP8 Stilprüfungen mit der Syntaxvalidierung des Modulnamens kombiniert, und recherchiere, wie man eine separate E Regel für die Syntaxprüfung erstellt.
Tech Stack
rustpython
Domain
clideveloper experience
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
RustPython
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60