sindresorhus/eslint-plugin-unicorn

`escape-case` breaks code

Chiusa

#2341 aperta il 8 mag 2024

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)JavaScript (468 fork)user submission
bughelp wanted

Metriche repository

Star
 (5022 stelle)
Metriche merge PR
 (Merge medio 1g 16h) (399 PR mergiate in 30 g)

Descrizione

Tagged template literals are not safe to change cases.

(function (a) {console.log(a.raw[0])})`\u{00b1}`
// -> \u{00b1}

(function (a) {console.log(a.raw[0])})`\u{00B1}`
// -> \u{00B1}

Guida contributor

Direzione di ricerca
Il problema è un bug nella regola `escape case` di eslint plugin unicorn. Attualmente la regola converte le sequenze di escape in minuscolo, ma questo rompe i letterali di template con tag in cui la stringa grezza è importante. Per risolverlo, la regola dovrebbe saltare i letterali di template con tag o trasformare solo le sequenze di escape all'interno dei letterali di stringa regolari. Il file sorgente rilevante è probabilmente `rules/escape case.js` e il suo file di test. Una correzione implicherebbe controllare se il letterale di stringa è un template con tag e bypassare la trasformazione.
Tech stack
javascript
Dominio
tooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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
JavaScriptESLint
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70