dotnet/roslyn

Fix VB Parsing of Format Specifiers

Offen

#58.043 geöffnet am 30.11.2021

Auf GitHub ansehenContributor Guide
 (16 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C# (4.257 Forks)batch import
Area-CompilersBugLanguage-VBhelp wanted

Repository-Metriken

Stars
 (20.414 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 6T 17h) (256 gemergte PRs in 30 T)

Beschreibung

VB has the same parsing "bug" as detailed in https://github.com/dotnet/roslyn/issues/57750, but unlike C# it does not support interpolated string handlers, so there are no runtime impacts to the parsing issue. We should update the VB parser to be in sync with the C# parser and the runtime here, but it's not high priority so the team is not going to focus on it. We'd accept a community PR to implement this fix.

Contributor Guide

Research-Richtung
Untersuche die C# Korrektur in Issue #57750, um die Parsing Änderung zu verstehen. Finde die entsprechende VB Parserlogik in der Roslyn Codebasis, wahrscheinlich in src/Compilers/VisualBasic/Portable/Parsing. Implementiere das gleiche Update für VB und stelle sicher, dass es keine Laufzeitauswirkungen hat, da VB keine Interpolated String Handler hat. Teste die Änderung mit den vorhandenen VB Parsing Tests.
Tech Stack
csharp
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75