E701: Move one-liner classes from E701 to E704?
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 42/100
Direzione di ricerca
Inizia nell'implementazione Python in un singolo file del repository, in compound_statements, e verifica come STARTSWITH_DEF_REGEX seleziona i casi E704. Conferma le diagnosi attuali per le classi e le funzioni su una sola riga, quindi stabilisci i messaggi E701/E704 attesi e il comportamento ignorato per impostazione predefinita. Il lavoro è completato quando la gestione concordata delle classi e la formulazione delle diagnosi sono coperte dai check del progetto.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- Python
- Stelle
- 5.2k
- Fork
- 756
- Merge medio
- 16h 56m
- PR unite (30g)
- 1
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di PyCQA/pycodestyle
-
DOC: E252 is not documentedAperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 82/100
PyCQA/pycodestyle#1294 · 1 commento ·
-
Typing and formattingAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
PyCQA/pycodestyle#1302 · 3 commenti ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 42/100
PyCQA/pycodestyle#1242 · 2 commenti ·
-
Spurious E231 for nested format substitutionsForse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
PyCQA/pycodestyle#1241 · 2 commenti · 4 reazioni ·
-
`exclude` is applied to `.`Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
PyCQA/pycodestyle#1217 ·
Tutte le issue di PyCQA/pycodestyle
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
-
EvaluationSuite.run fails with default args_for_task and mutates supplied kwargsForse già presa @ktz03 l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
huggingface/evaluate#825 ·
I maintainer di solito rispondono entro 1 giorno
-
dependencies feature github_actions good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
wemake-services/wemake-django-template#3149 ·
I maintainer di solito rispondono entro 1 giorno
-
[request] vsg/1.1.16Apertaupstream update
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
conan-io/conan-center-index#31142 ·
I maintainer di solito rispondono entro 1 giorno
-
area:core bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno