prettier/prettier

requirePragma doesn't work if @prettier pragma comes after other pragmas like @flow

Aperta

#5352 aperta il 5 nov 2018

Vedi su GitHubGuida contributor
 (4 commenti) (0 reazioni) (0 assegnatari)JavaScript (4983 fork)batch import
help wantedlang:javascript

Metriche repository

Star
 (52.203 stelle)
Metriche merge PR
 (Merge medio 6g) (186 PR mergiate in 30 g)

Descrizione

Environments:

  • Prettier Version: 1.14.3
  • Running Prettier via: atom prettier package

Steps to reproduce: (Put "requirePragma": true in .prettierrc)

// @flow
/**
 * @prettier
 */

     const    foo =    bar   +   baz

Expected behavior: Prettier formats the file

Actual behavior: Prettier doesn't format the file, unless I move the @prettier pragma to the top.

The problem

Any tools that require their pragma to be the very first comment will not work together. So to be a good citizen of the ecosystem, prettier must accept its pragma in any position among the file's leading comments.

Guida contributor

Direzione di ricerca
Trova la funzione che controlla il pragma @prettier (probabilmente nel modulo dei commenti) e modificala per scansionare tutti i commenti principali, non solo il primo. Quindi aggiorna i test corrispondenti.
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.
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
JavaScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75