luacheck with builtin minetest (#109)

* [PoC] luacheck with builtin minetest

* switch to `minetest+max` std for luacheck

* use branch in action

* use SX's docker image

* disable tty

* Update .github/workflows/luacheck.yml

Co-authored-by: SX <[email protected]>

* go back to upstream luacheck

* fix globals listing

---------

Co-authored-by: BuckarooBanzay <[email protected]>
Co-authored-by: SX <[email protected]>

.github/workflows/luacheck.yml

@@ -7,4 +7,4 @@ jobs:
       - name: Checkout
         uses: actions/checkout@master
       - name: Luacheck
-        uses: lunarmodules/luacheck@master
+        uses: lunarmodules/luacheck@master

.luacheckrc

@@ -1,26 +1,17 @@
 unused_args = false
 max_line_length= 240
 redefined = false
+std = "minetest+max"
 
 globals = {
 	"pipeworks",
 	"luaentity"
 }
 
 read_globals = {
-	-- Stdlib
-	string = {fields = {"split"}},
-	table = {fields = {"copy", "getn"}},
-
-	-- Minetest
-	"vector", "ItemStack",
-	"dump", "minetest",
-	"VoxelManip", "VoxelArea",
-
 	-- mods
 	"default", "mesecon", "digiline",
 	"screwdriver", "unified_inventory",
 	"i3", "mcl_experience", "awards",
-	"xcompat", "fakelib", "vizlib",
-
+	"xcompat", "fakelib", "vizlib"
 }