cleaner: add a rule against the void operator
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 82/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- typescript
- Ambito
- developer-experience
Direzione di ricerca
Inizia in scripts/cleaner.mjs all'array RULES e copia la struttura della voce no-types-in-components (id, description, fix, appliesTo, check). In check, usa ts.isVoidExpression(node) per trovare l'operatore void, e limita appliesTo ai file .ts e .tsx sotto src/renderer/src. Aggiungi un blocco describe in tests/cleaner.test.ts che copra un void load() segnalato e un tipo di ritorno (): void non segnalato. Fatto quando pnpm test passa, pnpm cleaner non riporta ancora nulla sul codice corrente, e l'elenco dei controlli in CONTRIBUTING.md riporta la nuova regola.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
pnpm cleaner checks the conventions in CONTRIBUTING.md that Prettier and TypeScript don't. It has one rule so far, and new ones get added one at a time. This issue adds the next: don't use the void operator (from the Code style section).
void load() // ✖ flagged
load() // ✔
const f = (): void => {} // ✔ the void *type* is fine
The renderer already follows this, so the new rule should pass on the current code.
Where to look
scripts/cleaner.mjs:RULES. Copy the shape ofno-types-in-components: anid, a one-linedescription, afix,appliesToandcheckts.isVoidExpression(node)finds the operator. Thevoidtype is a different node, so it won't be flaggedappliesTogets every file undersrc/renderer/src, including.cssand.html, so limit it to.tsand.tsxtests/cleaner.test.ts: add adescribefor the rule next to the existing one
Done when
void something()anywhere in a renderer.ts/.tsxfile is reported with its line and column- The
voidtype in annotations isn't pnpm cleanerstill reports no problems on the current code, andpnpm testpasses- The line in CONTRIBUTING.md that lists what
pnpm cleanerchecks mentions the new rule
Suggested pull request title: chore(cleaner): flag the void operator
New to OpenPaste? CONTRIBUTING.md covers setup, the code conventions and the checks to run before a pull request (pnpm typecheck, pnpm test, pnpm format, pnpm cleaner). Leave a comment if you're picking this up so nobody doubles up.
Generated by Claude Code
- Lingua principale
- TypeScript
- Stelle
- 0
- Fork
- 3
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un 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 alexsilver8/openpaste
-
enhancement good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
alexsilver8/openpaste#31 ·
-
enhancement good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 92/100
alexsilver8/openpaste#32 ·
-
bug good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
alexsilver8/openpaste#30 ·
-
bug good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
alexsilver8/openpaste#29 ·
-
bug good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
alexsilver8/openpaste#27 ·
Tutte le issue di alexsilver8/openpaste
Issue simili
-
priority:high
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
immich-app/immich#32426 ·
I maintainer di solito rispondono entro 1 giorno
-
documentation good first issue hacktoberfest help wanted
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
libredb/libredb-studio#1756 ·
I maintainer di solito rispondono entro 1 giorno
-
state:triage-needed
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
I maintainer di solito rispondono entro 1 giorno