[react/jsx-fragments] Enforce <Fragment> · jsx-eslint/eslint-plugin-react#2420

2019-09-26T14:34:42.000Z

I would like to enforce using ` ... ` instead of ` ... `. The fix option would mean to also add it to the imports if it hasn't been already. So these would be a warning: ```jsx import React from 'react'; export default Component = () => ( <>Foo ); ``` ```jsx import React from 'react'; export default Component = () => ( Foo ); ``` This would be the fix: (and should be fixable with `--fix`) ```jsx import React, { Fragment } from 'react'; export default Component = () => ( Foo ); ``` Is this achievable?

(7 commentaires) (12 réactions) (0 assignés)JavaScript (2 797 forks)batch import

help wantednew rule

Métriques du dépôt

Stars: (8 630 stars)
Métriques de merge PR: (Aucune PR mergée en 30 j)

Description

I would like to enforce using <Fragment>...</Fragment> instead of <React.Fragment>...</React.Fragment>. The fix option would mean to also add it to the imports if it hasn't been already.

So these would be a warning:

import React from 'react';

export default Component = () => (
    <>Foo</>
);

import React from 'react';

export default Component = () => (
    <React.Fragment>Foo</React.Fragment>
);

This would be the fix: (and should be fixable with --fix)

import React, { Fragment } from 'react';

export default Component = () => (
    <Fragment>Foo</Fragment>
);

Is this achievable?

Guide contributeur

Direction de recherche: Apprenez à créer une nouvelle règle dans eslint plugin react avec correction automatique qui transforme <React.Fragment> en <Fragment> et gère l'ajout d'import. Étudiez les règles existantes comme jsx no target blank pour les modèles.
Stack technique: javascriptreact
Domaine: developer experiencetooling
Type d'issue: Fonctionnalité
Difficulté: 3
Temps estimé: 1-2 jours
Statut d'activité: Active
Clarté: Claire
Prérequis: Node.jsESLintGit
Accessibilité débutant: 60

Métriques du dépôt

Description

Guide contributeur

Recevez de nouvelles issues Easy par e-mail.