`namespaces` rule doesn't respect scopes

Métriques du dépôt

Stars

5 940 étoiles (5 940 étoiles)

Métriques de merge PR

Merge moyen 138j 22h (Merge moyen 138j 22h)3 PRs mergées en 30 j (3 PRs mergées en 30 j)

Description

import React from 'react'
import * as wizards from './wizards'

// `wizards` redefined in local scope
function ComponentX({ wizards, submitWizard }) {
  return (
    <div>
      { shouldShowWizard && !isEmpty(wizards)
        // unable to validate computed reference report here at `wizards[0]`
        ? <WizardZero {...wizards[0]} onSubmit={submitWizard} /> 
        : <WhateverElse />
      }
    </div>
  )
}

Guide contributeur

Direction de recherche

Reproduisez le probleme en creant un cas de test avec un import de namespace (import * as mod) et en redefinissant le meme nom de variable comme parametre de fonction. Examinez l'analyse de portee de la regle pour garantir que la variable masquee ne provoque pas de faux positifs. La correction implique probablement de verifier si le namespace importe est masque par une variable locale.

Stack technique

javascript

Domaine

tooling

Type d'issue

Bug

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.

Une demi journée

Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.

Active

ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.

Claire

Prérequis

JavaScript

Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.

70