jsx-eslint/eslint-plugin-react

[Bug]: jsx-curly-newline and comments

Open

#3584 aperta il 7 giu 2023

Vedi su GitHub
 (6 commenti) (0 reazioni) (0 assegnatari)JavaScript (2797 fork)batch import
bughelp wanted

Metriche repository

Star
 (8630 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Is there an existing issue for this?

  • I have searched the existing issues and my issue is unique
  • My issue appears in the command-line and not only in the text editor

Description Overview

jsx-curly-newline reports an error when using multiline comments in JSX.

      {/* <FormGroup
        helperTextInvalid={helperText}
        inputProps={inputProps}
        type="password"
      /> */}

  134:7   error  Unexpected newline after '{'                             react/jsx-curly-newline
  138:12  error  Unexpected newline before '}'                            react/jsx-curly-newline

$ eslint --ext .js,.jsx,.ts,.tsx .

Expected Behavior

I would expect no error.

Rule options

"react/jsx-curly-newline": ["error", { "multiline": "forbid", "singleline": "forbid" }],

eslint-plugin-react version

7.32.2

eslint version

7.32.0

node version

14.21.3

Guida contributor

Direzione di ricerca
Comprendi come le regole di ESLint elaborano i nodi AST, in particolare JSXExpressionContainer. Esamina il codice sorgente della regola in eslint plugin react/lib/rules/jsx curly newline.js. Controlla come gestisce i commenti all'interno delle parentesi graffe JSX. La correzione probabilmente richiede di ignorare i commenti durante il calcolo delle posizioni delle nuove righe o di modificare la logica per consentire nuove righe attorno ai commenti su più righe.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
ESLint plugin development basicsAST (Abstract Syntax Tree) understanding
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40