sindresorhus/eslint-plugin-unicorn

`no-useless-fallback-in-spread`: Support ternary

Geschlossen

#1.861 geöffnet am 19.07.2022

Auf GitHub ansehenContributor Guide
 (4 Kommentare) (7 Reaktionen) (0 zugewiesene Personen)JavaScript (468 Forks)user submission
enhancementhelp wanted

Repository-Metriken

Stars
 (5.022 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 4h 30m) (26 gemergte PRs in 30 T)

Beschreibung

The rule currently does not detect ternary like this:

{...(foo ? {bar: true} : {})}

This can be safely be reduced to

{...(foo && {bar: true})}

Contributor Guide

Research-Richtung
Das Problem besteht darin, die Regel `no useless fallback in spread` zu erweitern, um ternäre Ausdrücke wie `{...(foo ? {bar: true} : {})}` zu erkennen und die Verwendung von `{...(foo && {bar: true})}` vorzuschlagen. Beginnen Sie mit der Untersuchung der bestehenden Regelimplementierung in `rules/no useless fallback in spread.js` und deren Tests. Prüfen Sie, wie sie derzeit verschiedene Muster behandelt. Die Lösung erfordert wahrscheinlich das Hinzufügen einer Prüfung für `ConditionalExpression` Knoten und die Sicherstellung, dass der Konsequente ein Objektausdruck und die Alternative ein leeres Objekt ist. Konsultieren Sie den ESLint Regelleitfaden für AST Knotentypen. Implementieren Sie die Logik und fügen Sie entsprechende Testfälle hinzu.
Tech Stack
javascript
Domain
toolingdeveloper experience
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Understanding of ASTFamiliarity with ESLint rule structureBasic JavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75