Add `err` and/or `opts` to the no-param-reassign exclusions?
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- eslint, javascript
- Ambito
- tooling
Direzione di ricerca
Esamina la configurazione della regola no-param-reassign e la discussione a cui si fa riferimento in #1089. Determina se opts e err, singolarmente o insieme, debbano essere esclusi e conferma il comportamento previsto per le assegnazioni di metadati come err.commandArgv e err.commandName; il completamento richiede una decisione approvata da un maintainer e la modifica corrispondente.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Hey there.
Most frequent i have problem with this rule. And specifically in cases of options, opts and err. I also seen in #1089 that this rules is disabled for different reasons. What's your thoughts? Could you consider adding opts and err?
I know that there is a bit better and more proper way for adding to the error instance, for example creating whole new error class based on the original one. But there is simple things like adding some single metadata property.
In my case such metadata is err.commandArgv and err.commandName where commandName isn't exactly the terminal/unix one, but command of some cli - for example mycli hello so err.commandName will be hello regardless what that command does.
- Lingua principale
- JavaScript
- Stelle
- 148k
- Fork
- 26.6k
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
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 airbnb/javascript
-
Severity: Unhandled promise rejection in `whitespace-async.js` when ESLint async path is usedForse già presa @bodapatisaikrishna l’ha presa 24 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
airbnb/javascript#3237 · 6 commenti ·
-
Inconsistent semicolon usage in examples (Arrays vs Functions)Forse già presa @Developer-shivamMishra l’ha presa 15 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
airbnb/javascript#3152 · 4 commenti ·
-
No error handling around execSync + JSON.parse in whitespace.js (ESLint 9 path)Forse già presa @dataCenter430 l’ha presa 219 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 48/100
airbnb/javascript#3238 · 8 commenti ·
-
Upgrading eslint-plugin-react-hooksForse già presa @weihongyu12 l’ha presa 365 giorni fa. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 38/100
airbnb/javascript#3186 · 3 commenti · 2 reazioni ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
airbnb/javascript#3173 · 9 commenti · 1 reazione ·
Tutte le issue di airbnb/javascript
Issue simili
-
type: bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
blamechris/chroxy#8579 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
modusensus/dsh-mneme#411 ·
-
Flaky (suspected): VideoTimelineEditor bed audio test sees 600px instead of 120px under CI loadAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno
-
Actions on ToastsAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
nextcloud-libraries/nextcloud-dialogs#2669 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
XRPLF/xrpl-dev-portal#4000 ·
I maintainer di solito rispondono entro 1 giorno