Add `err` and/or `opts` to the no-param-reassign exclusions?
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 35/100
- Issue-Typ
- Feature
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- eslint, javascript
- Bereich
- tooling
Rechercherichtung
Überprüfe die Konfiguration der no-param-reassign-Regel und die in #1089 referenzierte Diskussion. Ermittle, ob opts und err, entweder einzeln oder gemeinsam, ausgeschlossen werden sollten, und bestätige das erwartete Verhalten für Metadatenzuweisungen wie err.commandArgv und err.commandName; für den Abschluss ist eine von einem Maintainer genehmigte Entscheidung und die entsprechende Änderung erforderlich.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 148k
- Forks
- 26.5k
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Entwicklungsumgebung
Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.
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 airbnb/javascript
-
Severity: Unhandled promise rejection in `whitespace-async.js` when ESLint async path is usedEvtl. vergeben @bodapatisaikrishna hat das vor 43 Tagen übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
airbnb/javascript#3237 · 6 Kommentare ·
-
Inconsistent semicolon usage in examples (Arrays vs Functions)Evtl. vergeben @Developer-shivamMishra hat das vor 33 Tagen übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
airbnb/javascript#3152 · 4 Kommentare ·
-
No error handling around execSync + JSON.parse in whitespace.js (ESLint 9 path)Evtl. vergeben @dataCenter430 hat das vor 238 Tagen übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 48/100
airbnb/javascript#3238 · 8 Kommentare ·
-
Upgrading eslint-plugin-react-hooksEvtl. vergeben @weihongyu12 hat das vor 384 Tagen übernommen. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 38/100
airbnb/javascript#3186 · 3 Kommentare · 2 Reaktionen ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
airbnb/javascript#3173 · 9 Kommentare · 1 Reaktion ·
Alle Issues in airbnb/javascript
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 85/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 73/100
microsoft/monaco-editor#5563 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 85/100
Maintainer antworten meist innerhalb von 1 Tag
-
account: profile bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Maintainer antworten meist innerhalb von 5 Tagen
-
ai-driven-fuzzing bug claude Minor
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
linagora/cozy-client#1777 · 1 Kommentar · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag