Skip to content

Fixes, Changes, Add missing - #555

Merged
duckdoom4 merged 8 commits into
renpy:masterfrom
Ho11ow1:master
Jul 26, 2026
Merged

duckdoom4 merged 8 commits into
renpy:masterfrom
Ho11ow1:master

Conversation

@Ho11ow1

@Ho11ow1 Ho11ow1 commented Jul 23, 2026

Copy link
Copy Markdown
Contributor
  • Updated persistent variable regex to be able to catch variables containing under_scores
  • Fixed a typo in the scripts rst README
  • Fixed formatting across multiple python files
  • Added missing test and knot tokens
  • Modified TS indicy check to correctly check indicies and no longer need an extra array index
  • Modify generate.py to now generate tokenizer/generated every time rather than only when syntaxes/json | yaml as the file changed check was unnecessary and only blocked builds (At least from my testing)

@duckdoom4

Copy link
Copy Markdown
Collaborator

Looks good to me!

Just one formatting error and 2 linter warnings

@duckdoom4

duckdoom4 commented Jul 26, 2026

Copy link
Copy Markdown
Collaborator

As a side note:
Have a peak at the parser branch.
With that version we can implement all the features you'd expect of a language plugin. It is in functional state, but still requires some work to include all the renpy features before we can publish it.
AI is basically perfect for that job, as it's literally just a matter of converting renpy's syntax parser logic into rules.

@duckdoom4
duckdoom4 self-requested a review July 26, 2026 15:05
@duckdoom4 duckdoom4 self-assigned this Jul 26, 2026
@duckdoom4
duckdoom4 merged commit 659ba11 into renpy:master Jul 26, 2026
4 checks passed
@Ho11ow1

Ho11ow1 commented Jul 26, 2026

Copy link
Copy Markdown
Contributor Author

Thanks for responding,
I'll take a look at the parser branch and see how everything looks over there.

Not sure if my efforts over there will bare any fruit but i guess we'll just have to wait and see, might be interesting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants