microsoft / microsoft/node-jsonc-parser
Feature request: Support (optional) bigint during parsing
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 759
- Forks
- 66
- Ø Merge
- 5 T. 10 Std.
- Gemergte PRs (30 T.)
- 7
Beschreibung
Currently, number literal will be parsed by the following code:
It would be better to add a feature that allow detecting the number and parsing it with BigInt, like:
// -9007199254740991 ~ 9007199254740991
// More precise checking could be used, here's only a demo
if (/^-?\d{16,}$/.test(tokenValue)) {
value = BigInt(tokenValue)
}
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne bei src/impl/parser.ts um die Zeilen 515-519, wo Zahlenliterale derzeit geparst werden. Verfolge, wie geparste Werte verfügbar gemacht werden, und bestimme, wo ein optionales bigint-Verhalten konfiguriert werden könnte. Als abgeschlossen gilt die Änderung, wenn große Integer-Literale auf Anfrage mit BigInt geparst werden können, ohne das standardmäßige Parsing-Verhalten zu ändern.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- typescript
- Bereich
- tooling
- Issue-Typ
- Feature
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100