reactjs / reactjs/react-docgen

Type annotation with flow syntax and defaulting of args is breaking React Class validator

Offen
#284 5 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

needs-more-info
Vorherrschende Sprache
TypeScript
Sterne
3.8k
Forks
316
Ø Merge
5 Std. 7 Min.
Gemergte PRs (30 T.)
4

Beschreibung

Hello,

Thanks for your library!

I ran into an issue where a option (flow annotation) argument in a handler was causing my class to not be parsed.. resulting in You have to export at least one valid React Class!. Defaulting the arg also seems to break things.

Breaks:

handleThings = (id: number, active?: boolean) => () => {
   things..
}
handleThings = (id: number, active: boolean = false) => () => {
   things..
}

Works:

handleThings = (id: number, active: boolean) => () => {
   things..
}

Thanks!

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne beim React Class validator und reproduziere die beiden fehlschlagenden JSX-Beispiele zusammen mit dem funktionierenden Beispiel. Verfolge, warum optionale Flow-Annotationen und Standardwerte von Argumenten verhindern, dass die Klasse erkannt wird; fertig ist die Aufgabe, wenn alle drei Formen als gültige React-Klassen geparst werden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript, react
Bereich
tooling
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.