Clear the unused-variable lint warnings in `census-intake.ts`
Una pull request collegata è già stata integrata.
Valutazione
- Difficoltà
- 1/5
- Tempo stimato
- Meno di un'ora
- Idoneità per principianti
- 90/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- typescript
- Ambito
- frontend
Direzione di ricerca
In web/, esamina src/lib/intake.ts intorno alla riga 239 ed esegui npm run lint per confermare l’avviso. Rendi anyRecord una dichiarazione di solo tipo oppure ricava il tipo in un altro modo; il lavoro è concluso quando lint non segnala più quell’avviso.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
npm run lint in web/ warns twice:
./src/lib/census-intake.ts
55:17 Warning: '_guid' is assigned a value but never used. @typescript-eslint/no-unused-vars
55:33 Warning: '_clientEventId' is assigned a value but never used. @typescript-eslint/no-unused-vars
The line drops two fields from a row with a destructuring rest (const { guid: _guid, eventId: _clientEventId, ...rest } = row).
The names are discarded on purpose, but the lint rule in web/.eslintrc.js only ignores _-prefixed
function arguments, not variables.
(This issue first named an anyRecord warning in src/lib/intake.ts; that code has since moved to
packages/protocol and the warning is gone. These two are what is left.)
Done when
cd web && npm run lintprints no warnings.- The fix is either a lint setting (for example
ignoreRestSiblings: trueorvarsIgnorePattern: "^_"
on@typescript-eslint/no-unused-vars) or a change to the one line. Say in the pull request which
you picked and why. npm testinweb/still passes;web/src/lib/census-intake.test.tscovers the function.
- Lingua principale
- TypeScript
- Stelle
- 0
- Fork
- 0
- 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 CDVolvik/everloot
-
data help wanted
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
-
Fewer damaged lore pagesApertahelp wanted lore
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
-
Architecture north star: one data layer, API tiers, contributions, addon directory, ads, appsAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 20/100
-
data help wanted
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
-
addon help wanted
Difficoltà 4/5 3-5 giorni Idoneità per principianti 58/100
Tutte le issue di CDVolvik/everloot
Issue simili
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
awslabs/visual-asset-management-system#804 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
restatedev/sdk-typescript#813 ·
-
bug desktop P1 session XS
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
Merit-Systems/x402scan#1312 ·
-
type:bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
Azure/apiops-cli#331 ·
I maintainer di solito rispondono entro 3 giorni