E701: Move one-liner classes from E701 to E704?
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 42/100
Rechercherichtung
Beginne in der einteiligen Python-Implementierung des Repositorys bei compound_statements und untersuche, wie STARTSWITH_DEF_REGEX die E704-Fälle auswählt. Bestätige die aktuellen Diagnosen für einzeilige Klassen und Funktionen und lege anschließend die erwarteten E701/E704-Meldungen sowie das standardmäßig ignorierte Verhalten fest. Erledigt ist die Aufgabe, wenn die vereinbarte Klassenbehandlung und die Diagnoseformulierung durch die Checks des Projekts abgedeckt sind.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Currently, one-liner classes (class Foo: pass or for example more realistically class T(Enum): a, b, c = range(3)) trigger E701 ("multiple statements on one line (colon)").
I would suggest moving them to E704 ("multiple statements on one line (def)") (changing the message to use "def/class") as I think a one-liner class is closer in spirit to a one-liner function than to constructs like if cond: ... or for ... in ...: ....
This would also make this check ignored by default.
I think the implementation is simply a matter of changing the use of STARTSWITH_DEF_REGEX in compound_statements to, well also match class statements; would be happy to provide a PR if the change is agreed upon.
- Vorherrschende Sprache
- Python
- Sterne
- 5.2k
- Forks
- 756
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine 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 PyCQA/pycodestyle
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 82/100
PyCQA/pycodestyle#1294 · 1 Kommentar ·
-
Typing and formattingOffen
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100
PyCQA/pycodestyle#1302 · 3 Kommentare ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 42/100
PyCQA/pycodestyle#1242 · 2 Kommentare ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 68/100
PyCQA/pycodestyle#1241 · 2 Kommentare · 4 Reaktionen ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
PyCQA/pycodestyle#1217 ·
Alle Issues in PyCQA/pycodestyle
Ähnliche Issues
-
repo-audit
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
scverse/repo-health#20 ·
Maintainer antworten meist innerhalb von 1 Tag
-
/context/prime scope override double-prefixes an entity-ref project and drops its scoped memoriesOffen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 85/100
phasespace-labs/palinode#232 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
collective/icalendar#1858 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
langflow-ai/langflow#15496 ·
Maintainer antworten meist innerhalb von 1 Tag