atom/github

Replace getDerivedStateFromProps with memoization

Open

#1.611 geöffnet am 26. Juli 2018

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (469 Forks)batch import
code-cleanup :wrench:good first issue

Repository-Metriken

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

Beschreibung

Where possible, it would be cleaner and less error-prone to replace components that implement static getDerivedStateFromProps() with PureComponents or memoize-one:

Contributor Guide

Research-Richtung
Finde alle React Komponenten, die static getDerivedStateFromProps verwenden. Bestimme für jede Komponente, ob sie in einen PureComponent umgewandelt werden kann oder ob Memoization mit memoize one geeignet ist. Befolge die Anleitung im verlinkten Blogbeitrag, um die Lebenszyklusmethode zu ersetzen. Führe die vorhandenen Tests aus, um sicherzustellen, dass keine Regressionen auftreten.
Tech Stack
reactjavascript
Domain
frontend
Issue Type
Refactoring
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.
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
ReactJavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75