exercism / lua

Exercism exercises in Lua.
https://exercism.org/tracks/lua
MIT License
41 stars 58 forks source link

Add spec generator and script for installing development dependencies #504

Closed ryanplusplus closed 2 months ago

ryanplusplus commented 2 months ago

This adds an optional spec generator that makes it easier to create and maintain spec files. There's no tooling that enforces its use and, in my experience on other tracks, some exercises are challenging/awkward to generate specs for so I see this as a feature. I included the configuration that I used to generate the wordy spec.

Since the generator depends upon dkjson and luaformatter, I also added a rockspec that lists the development dependencies and a script to use LuaRocks to install these dependencies.

Finally, I documented the spec generator and development dependency installation in the README.

Closes https://github.com/exercism/lua/issues/259

github-actions[bot] commented 2 months ago

This PR touches files which potentially affect the outcome of the tests of an exercise. This will cause all students' solutions to affected exercises to be re-tested.

If this PR does not affect the result of the test (or, for example, adds an edge case that is not worth rerunning all tests for), please add the following to the merge-commit message which will stops student's tests from re-running. Please copy-paste to avoid typos.

[no important files changed]

For more information, refer to the documentation. If you are unsure whether to add the message or not, please ping @exercism/maintainers-admin in a comment. Thank you!

ryanplusplus commented 2 months ago

This PR touches files which potentially affect the outcome of the tests of an exercise. This will cause all students' solutions to affected exercises to be re-tested.

If this PR does not affect the result of the test (or, for example, adds an edge case that is not worth rerunning all tests for), please add the following to the merge-commit message which will stops student's tests from re-running. Please copy-paste to avoid typos.

[no important files changed]

For more information, refer to the documentation. If you are unsure whether to add the message or not, please ping @exercism/maintainers-admin in a comment. Thank you!

This is because the capitalization of a test name was changed by the generator. We can safely merge with [no important files changed].