search

lcpz / lain

Awesome WM complements
GNU General Public License v2.0
1.06k stars 210 forks source link

Automated testing with Luacheck #533

Closed thomascft closed 2 years ago

thomascft commented 2 years ago

This is my fix for #527. Well, I don't know git well enough clearly. I setup the testing, you can see the output here. I had a bit of cleanup, so that's what the force pushes were

thomascft commented 2 years ago

I should probably mention that the test rules are defined in .luacheckrc with the documentation over here.

lcpz commented 2 years ago

Thank you for this. However, we still need unit tests.

It's not so difficult now, we just need to create a test script with luaunit and run it in the Action you created, just after the linter call.

Feel free to continue to help me, I think it's a nice exercise for you.

Otherwise, I will work on it as soon as I can.