jsx-eslint/eslint-plugin-react

[jsx-key]: report on `yield`

Open

#352 ouverte le 7 déc. 2015

Voir sur GitHub
 (2 commentaires) (1 réaction) (0 assignés)JavaScript (8 630 stars) (2 797 forks)batch import
enhancementhelp wantedrule

Description

Seems like yield <Component x={y}/> should be reported by jsx-key if there is no key prop.

This is valid:

for (let [idx, datum] of data.entries()) {
  yield (<MyComponent datum={datum} key={idx} />)
}

This is reported:

for (let [idx, datum] of data.entries()) {
  yield (<MyComponent datum={datum} />) // msg: need `key` prop
}

I can knock this out, if it makes sense.

(cc #332 as it will likely conflict with this if completed concurrently.)

Guide contributeur

Stack technique
javascriptreact
Domaine
developer experiencetooling
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
half day
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
ESLint plugin developmentReact JSXAST traversalJavaScript generators
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75
Direction de recherche
Examine the jsx key rule implementation in the eslint plugin react repository to understand how it currently detects JSX elements without key props. Add handling for JSX elements inside yield expressions, similar to how it handles for loop and array map blocks. Consider potential conflicts with issue #332 which may also modify the same area. Write tests to cover yield in different contexts.