withastro/astro

🐛 BUG: version 2.8.1 removes trailing comma on save

Chiusa

#14.665 aperta il 29 mar 2024

Vedi su GitHubGuida contributor
 (7 commenti) (0 reazioni) (0 assegnatari)TypeScript (3719 fork)batch import
good first issuepkg: language-tools

Metriche repository

Star
 (61.815 stelle)
Metriche merge PR
 (Merge medio 4g 23h) (197 PR mergiate in 30 g)

Descrizione

Describe the Bug

When I save an astro file, it removes trailing commas from multi-line function calls.

If I downgrade to version 2.8.0 and save, it adds them back again when I save.

I wonder if it is also related to withastro/language-tools#823 - I found another issue that sounded like it was similarly affected, withastro/language-tools#829 - though the specific fix for that one did not address this one.

Steps to Reproduce

  1. Open any .astro file with multi-line function calls. E.g. 
    foo(
  var1,
  var2,
);
  2. save the file
  3. trailing commas are removed. For the above example it would change to: 
    foo(
  var1,
  var2
);

EDIT: After comments, here is a better example, since the above ends up on a single line which does not have trailing comma:

  someReallyLongNAmeThingThatIsReallyLong(
    someReallyLongVarThatISLongAnd,
    SomeOtherReallyLongThing,
  );

Guida contributor

Direzione di ricerca
Analizza la logica di formattazione del server linguistico di Astro, in particolare nel plugin Prettier o nel formattatore interno. Esamina i problemi segnalati withastro/language tools#823 e #829 per il contesto correlato. Concentrati su come vengono gestite le virgole finali nelle chiamate di funzione su più righe all'interno dei file .astro. Il bug sembra essere una regressione nella versione 2.8.1, quindi confronta il codice di formattazione tra le versioni 2.8.0 e 2.8.1.
Tech stack
typescriptnodejs
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.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
TypeScriptAstro
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50