Merge remote-tracking branch 'upstream/master'

Fixed the error where it cannot find an user at the notice hook. 
It happened when you get the `NOTICE * :*** Looking up your hostname and checking ident`
Because `*` is not in the `<nick>!<ident>@<host>`

.github/workflows/check-release.yml

@@ -0,0 +1,11 @@
+on: [push, pull_request]
+name: Check & Release
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@master
+      - name: lint
+        uses: Roang-zero1/factorio-mod-luacheck@master
+        with:
+          luacheckrc_url: https://raw.githubusercontent.com/minetest-mods/irc/master/.luacheckrc

.luacheckrc

@@ -2,7 +2,7 @@
 allow_defined_top = true
 
 read_globals = {
-	"minetest",
+	"minetest"
 }
 
 exclude_files = {

README.md

@@ -1,3 +1,4 @@
+[![](https://github.com/minetest-mods/irc/workflows/Check%20&%20Release/badge.svg)](https://github.com/minetest-mods/irc/actions)
 
 IRC Mod for Minetest
 ====================

botcmds.lua

@@ -169,8 +169,10 @@ irc.register_bot_command("players", {
 		for _, player in pairs(players) do
 			table.insert(names, player:get_player_name())
 		end
-		return true, "Connected players: "
+		return true, string.format("%d connected player(s): %s",
-				..table.concat(names, ", ")
+			#players,
+			table.concat(names, ", ")
+		)
 	end
 })
 

callback.lua

@@ -31,7 +31,7 @@ minetest.register_on_chat_message(function(name, message)
 	if nl then
 		message = message:sub(1, nl - 1)
 	end
-	irc.say(irc.playerMessage(name, core.strip_colors(message)))
+	irc.say(irc.playerMessage(name, minetest.strip_colors(message)))
 end)
 
 

hooks.lua

@@ -171,7 +171,7 @@ end
 
 
 function irc.hooks.notice(user, target, message)
-	if user.nick and target == irc.config.channel then
+	if user and user.nick and target == irc.config.channel then
 		irc.sendLocal("-"..user.nick.."@IRC- "..message)
 	end
 end

player_part.lua

@@ -47,7 +47,7 @@ minetest.register_chatcommand("who", {
 			out[n] = plname
 		end
 		table.sort(out)
-		return true, "Players in channel: "..table.concat(out, ", ")
+		return true, n.." player(s) in channel: "..table.concat(out, ", ")
 	end
 })