ChoicescriptIDE / ChoicescriptIDE/main

Warn/lint users about potential for recursive issue with *text_image

Offen
#159 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

needs triage
Vorherrschende Sprache
JavaScript
Sterne
36
Forks
8
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

https://forum.choiceofgames.com/t/new-choicescript-extension-for-vs-code-2-0-now-runs-games/71387/73?u=cjw

I recently discovered a new bug that can cause recursion issues once a game is compiled. If you were to use the *text_image command, but leave out the alignment and caption, it will cause a recursion error to occur once the data URI is implemented post-compilation.

For example, the following line will cause a recursion error after compiling the game:

*text_image sample.png
The alignment and caption needs to be included in order to avoid the recursion bug:

*text_image sample.png center Sample Caption
There’s not much information surrounding the *text_image command from what I can find. Yet, I was thinking perhaps it would be a nice addition to the extension if it notified the author/developer of the improper command. To save some future headaches if possible, as I was at a lost on what was causing the recursion bug initially.

@CJW Apologies for the tag, but this may also be something that could be added to the IDE, if it hasn’t been already?

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne damit, die Behandlung des *text_image-Befehls durch die Erweiterung sowie den Einstiegspunkt für die Kompilierung oder Diagnose zu finden. Reproduziere den gezeigten Befehl ohne Ausrichtung und Beschriftung, füge dann eine Warnung für diese Form hinzu und überprüfe, dass die vollständige Form weiterhin akzeptiert wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript
Bereich
tooling
Issue-Typ
Feature
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.