Closed robotboy655 closed 6 months ago
Heya @robotboy655 , thanks again for your work.
I see a test is failing. If it's only two or so tests, I'm fine with you commenting them for now, instead of writing new ones. I will work on writing some better tests for your code another time.
Let me know if you think this is good to merge and I'll get on it
@luttje I have fixed the tests.
I have tested the resulting output from my PR against a the previous version, and here's the result:
Basically only the documentation that was overwritten in custom/
has changed, so yeah, I think this is safe to merge.
This PR does the following:
It partially addresses issues in https://github.com/luttje/glua-api-snippets/issues/47 and reduces reliance on hardcoded overrides.
We do lose some nuance with callback types, such as
string[]
being replaced withtable
inconcommand.Add
autocomplete callback, but I think we can live with that.