-
-
Notifications
You must be signed in to change notification settings - Fork 201
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding LUA bindings #860
base: master
Are you sure you want to change the base?
Adding LUA bindings #860
Conversation
Fixes software supply chain safety warnings like at the bottom right of https://github.com/fairy-stockfish/Fairy-Stockfish/actions/runs/12274061623 * [Keeping your actions up to date with Dependabot](https://docs.github.com/en/code-security/dependabot/working-with-dependabot/keeping-your-actions-up-to-date-with-dependabot) * [Configuration options for the dependabot.yml file - package-ecosystem](https://docs.github.com/en/code-security/dependabot/dependabot-version-updates/configuration-options-for-the-dependabot.yml-file#package-ecosystem)
Thanks for the PR. There definitely is some value in having additional language bindings, but I currently do not see a concrete use case that would justify the maintenance overhead this creates, especially because it introduces git submodules to this project, and also given my lack of familiarity with Lua. So for now I am very hesistant about incorporating it. |
Fair enough - I plan to use them in a project I’m working on and opened the PR in case others might find them valuable. No problem if they’re not merged. They’ll be here if it turns out you’d prefer it to be part of main-line. |
The Lua bindings provide a powerful interface to the chess variant engine. They require Lua 5.1 for LuaJIT compatibility. See tests/lua/README.md for installation and usage instructions.