`namespaces` rule doesn't respect scopes

Metriche repository

Star

5940 stelle (5940 stelle)

Metriche merge PR

Merge medio 138g 22h (Merge medio 138g 22h)3 PR mergiate in 30 g (3 PR mergiate in 30 g)

Descrizione

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>
  )
}

Guida contributor

Direzione di ricerca

Riprodurre il problema creando un caso di test con un import namespace (import * as mod) e ridefinendo lo stesso nome di variabile come parametro di funzione. Esaminare l'analisi dello scope della regola per garantire che la variabile oscurata non causi falsi positivi. La correzione probabilmente implica verificare se il namespace importato è oscurato da una variabile locale.

Tech stack

javascript

Dominio

tooling

Tipo issue

Bug

DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.

3

Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.

Mezza giornata

Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.

Attiva

ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.

Chiara

Prerequisiti

JavaScript

Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.

70