import-js/eslint-plugin-import

Make optional reporting of "import/export: Multiple exports of name ..."

Open

#2 167 ouverte le 27 juil. 2021

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)JavaScript (1 540 forks)batch import
enhancementhelp wanted

Métriques du dépôt

Stars
 (4 946 stars)
Métriques de merge PR
 (Merge moyen 138j 22h) (3 PRs mergées en 30 j)

Description

Multiple exports of name is always reported as an error but it's a legitimate feature disciplined by ECMASCRIPT specs.

Es:

// D.js 
const f = () => 'D::f'
export { f }

// E.js
export * from './D.js'
const f = () => 'E::f'
export { f }

Report -> import/export: Multiple exports of name 'f'.

Maybe eslint-plugin-import could give an optionable error that can ben disabled by config?

See also here and #1704

Guide contributeur

Direction de recherche
Le problème demande de rendre l'erreur 'Multiple exports of name' optionnelle via une option de configuration. La règle concernée se trouve dans le dépôt eslint plugin import, probablement dans src/rules/export.js ou similaire. Le contributeur devra ajouter une nouvelle option (par exemple 'allowMultipleExports') et modifier la logique qui signale l'erreur. Référez vous à l'issue #1704 et au lien StackOverflow pour le contexte. La base de code utilise JavaScript et Node.js.
Stack technique
javascript
Domaine
developer experiencetooling
Type d'issue
Fonctionnalité
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
JavaScriptESLint
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40