import-js/eslint-plugin-import

Support Flow Enums

Aperta

#2073 aperta il 14 mag 2021

Vedi su GitHubGuida contributor
 (8 commenti) (4 reazioni) (0 assegnatari)JavaScript (1549 fork)batch import
enhancementflowhelp wanted

Metriche repository

Star
 (5940 stelle)
Metriche merge PR
 (Merge medio 138g 22h) (3 PR mergiate in 30 g)

Descrizione

Babel added support for flow enums here: https://github.com/babel/babel/pull/10344

Currently trying to import an enum results in MyEnum not found in enum-module

They're currently experimental, but should be coming down the pipe. I think it would be pretty trivial to support these, basically add an EnumDeclaration everywhere there is currently a TSEnumDeclaration.

I tried working on a PR for this, but the problem is getting the tests running. EnumDeclaration was added in babel 7.7.0. I was trying to get @babel/eslint-parser going in all the flow-related tests while retaining babel-eslint in other tests, but I kept running into problems.

Guida contributor

Direzione di ricerca
Aggiungi la gestione di EnumDeclaration insieme a TSEnumDeclaration nella logica di risoluzione delle importazioni del plugin. Assicurati di impostare correttamente i test con @babel/eslint parser per quelli relativi a Flow.
Tech stack
javascript
Dominio
tooling
Tipo issue
Funzionalità
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitNode.jsbabel
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70