jsx-eslint/eslint-plugin-react

require-default-props: Allow disabling "must have default value" warning but keeping "forbid defaults for required props"

Open

#3.660 geöffnet am 1. Dez. 2023

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (2.797 Forks)batch import
enhancementhelp wanted

Repository-Metriken

Stars
 (8.630 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Right now there doesn't seem to be a way to do this.

  • Having to specify a null/undefined default for a prop where you're perfectly fine with it being undefined when not specified feels a bit pointless
  • This is made worse for a prop where you want to allow null, because a required prop will fail runtime proptypes validation when its value is null (even if it's explicitly set to null

So I'd like to just ditch the part of having to specify default for optional props, but still prohibit setting defaults for required ones.

Contributor Guide

Research-Richtung
Untersuchen Sie die aktuelle Implementierung der require default props Regel in der eslint plugin react Codebasis (wahrscheinlich src/rules/require default props.js). Das Ziel ist es, eine neue Option hinzuzufügen, die die Warnung für fehlende Standardwerte bei optionalen Props deaktiviert, während die Prüfung, die Standardwerte bei erforderlichen Props verbietet, beibehalten wird. Überprüfen Sie vorhandene Testdateien (z. B. tests/rules/require default props.js), um das erwartete Verhalten zu verstehen und Abwärtskompatibilität sicherzustellen. Berücksichtigen Sie das Konfigurationsschema und wie die neue Option in bestehende Regeloptionen integriert werden kann.
Tech Stack
javascripttypescriptreact
Domain
frontenddeveloper experience
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
ESLintReact
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60