Conversation
|
Hey oliver Hey! Nice work on this — looks like you put a lot of effort into the struct generation and method mapping. The filtering logic makes sense, and it's cool to see how you're handling the SplashKit bindings. Great job 👍 |
Mdanial01
left a comment
There was a problem hiding this comment.
Overall, this is a strong and well-focused pull request. Introducing a generator based on api.json to produce C# bindings is a solid improvement that will significantly reduce manual maintenance and improve long-term consistency.
Before merging, I recommend addressing a few points:
Please investigate and remove any hidden or bidirectional Unicode characters flagged by GitHub.
Since a generated file is committed, it would be good to document the generation process and ensure it is deterministic (ideally with a CI check to prevent drift).
The hard-coded path in CSharpCodeRunner.cs should be confirmed to be correct for all environments or made configurable to avoid future maintenance issues.
Once these points are addressed, this PR will be a valuable and maintainable addition to the project.
|

Description
In this PR, I have added a code generator that works automatically with the
api.jsonto generate the bindings for C#.Type of change
How Has This Been Tested?
I have tested the output script with the current generation pipeline with no errors. I have then tested the output in SplashKit Online with C#
Testing Checklist
Checklist