streamich/react-use

Remove warnings from `useDeepCompareEffect`

Offen

#755 geöffnet am 06.11.2019

Auf GitHub ansehenContributor Guide
 (8 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (3.273 Forks)batch import
buggood first issuehelp wanted

Repository-Metriken

Stars
 (43.979 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

Is your feature request related to a problem? Please describe.

useDeepCompareEffect gives warnings when using it with primitive dependencies. This is frustrating for the common case where dependencies could either undefined or an object that you'd like to be deep-compared.

Describe the solution you'd like

Remove warnings from useDeepCompareEffect or provide a way to suppress warnings.

Contributor Guide

Research-Richtung
Finde die Implementierung von useDeepCompareEffect im Quellcode (wahrscheinlich in src/useDeepCompareEffect.ts). Untersuche die Warnlogik und entscheide, ob sie entfernt oder eine Unterdrückungsoption hinzugefügt werden soll. Lies die Issue Kommentare für Anleitungen. Stelle sicher, dass Änderungen bestehende Tests nicht brechen.
Tech Stack
typescriptreact
Domain
frontend
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
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.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
React hooks
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70