Consider returning specifically TSQLScript object from parser instead of abstraction
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 45/100
Rechercherichtung
Beginne mit der obersten Parse-Methode in SqlScriptDom/Parser/TSql/TSqlParser.cs und der internen Parse-Methode in TSql150Parser.cs, die im Issue verlinkt sind. Verfolge ihre Rückgabetypen und die Verarbeitung des Ergebnisses durch die HotSwap-Methode. Die Änderung ist abgeschlossen, wenn der Parser konsistent das konkrete TSQLScript-Ergebnis bereitstellt, sodass Consumer keinen Hard Cast benötigen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Since you know specfically that the result is TSQLScript why turning it into a less comfortable to work with abstraction for consumers?
To start working with TSQLScript and his batches one has to double-check or to perform a hard cast to TSQLScript just as you do in HotSwap method - why? Since it always was a TSQLScript instance.
- Vorherrschende Sprache
- GAP
- Sterne
- 278
- Forks
- 46
- Ø Merge
- 9 T. 23 Std.
- Gemergte PRs (30 T.)
- 2
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus microsoft/SqlScriptDOM
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
microsoft/SqlScriptDOM#228 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
microsoft/SqlScriptDOM#183 ·
-
Add a Multiline option for CASE expressions (WHEN/THEN/ELSE on their own lines)Evtl. vergeben @trg-alasdair hat das vor 7 Tagen übernommen. Offen
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 56/100
microsoft/SqlScriptDOM#226 · 1 Reaktion ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100
microsoft/SqlScriptDOM#225 ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 58/100
microsoft/SqlScriptDOM#224 ·
Alle Issues in microsoft/SqlScriptDOM
Ähnliche Issues
-
Slack progress shows a tool row for message react calls on the embedded runtimeEvtl. vergeben @obviyus hat das heute übernommen. Offenclawsweeper:fix-shape-clear clawsweeper:queueable-fix clawsweeper:source-repro impact:ux-friction issue-rating: 🦞 diamond lobster no-stale P2
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
openclaw/openclaw#166859 · 2 Kommentare · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag
-
backend bug size:sm
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
chrisbenincasa/tunarr#2237 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
bmander/geomsolver#118 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug needs-triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
uoftblueprint/canada-basketball#32 ·
Maintainer antworten meist innerhalb von 1 Tag