scroll root parser evalMacros should escape dollar signs in the replacement string
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 48/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- javascript
- Ambito
- compilers
Direzione di ricerca
Inizia da parsers/root.parsers intorno alla riga 835, dove evalMacros usa macroMap[key] come stringa di sostituzione per un'espressione regolare. Usa l'esempio collegato di PLDB/djot come caso di regressione; il lavoro è completato quando i simboli del dollaro letterali nell'output della macro vengono preservati invece di essere interpretati dalla sintassi JavaScript delle stringhe di sostituzione.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
As mentioned in this pldb issue, the djot page has messed up output.
I think the issue is the pldb example contains
$`p = mv`
and this line is using that in the regexp replacement string
https://github.com/breck7/scroll/blob/b8555981ad6bd39cb16b1ab766daf129ec0305b8/parsers/root.parsers#L835
According to MDN, the dollar sign followed by backtick has special meaning in the replacment string.
I think prior to using the replacement string (i.e. macroMap[key]), any dollar sign should be replaced with a double dollar sign.
- Lingua principale
- JavaScript
- Stelle
- 484
- Fork
- 21
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Non abbiamo ancora controllato i file di configurazione di questo progetto. Parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di breck7/scroll
-
Scroll.pub website is downAperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 10/100
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 20/100
-
Union atom typesAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 30/100
Tutte le issue di breck7/scroll
Issue simili
-
good first issue
Difficoltà 2/5 3-5 giorni Idoneità per principianti 62/100
paipai1999/POS-system#3 ·
-
good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
r-afael/better-yt-recommendations#36 ·
I maintainer di solito rispondono entro 1 giorno
-
needs-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
microsoft/playwright#43875 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
cli good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
I maintainer di solito rispondono entro 1 giorno
-
Run the dependency drift checkForse già presa @spaceninja l’ha presa oggi. Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 82/100
cloudfour/stylelint-config-cloudfour#683 ·
I maintainer di solito rispondono entro 1 giorno