Update luacheck rules

.drone.yml

@@ -19,7 +19,8 @@ steps:
     - apk add --no-cache build-base curl unzip
     - apk add --no-cache lua5.1 lua5.1-dev luarocks5.1
     - luarocks-5.1 install luacheck
-    - luacheck sources/*.lua --compat --allow-defined --no-unused-args --no-max-line-length
+    - cd sources
+    - luacheck ./*.lua
 
 - name: build
   image: alpine

.luacheckrc

@@ -0,0 +1,12 @@
+allow_defined = true
+max_line_length = false
+
+std = {
+   read_globals = {
+       "LINK_STYLE_BRACKET", 
+       "EVENT_MANAGER", 
+       "CHAT_SYSTEM", 
+       "ZO_SavedVars", 
+       "LibStub"
+    }
+}

AchievementInfoApplication.lua

@@ -50,7 +50,6 @@ function AchievementInfo.onAchievementUpdated(eventCode, achId)
     local percentageStepSize    = AchievementInfo.settingGet("genShowUpdateSteps")
 
     local link = GetAchievementLink(achId, LINK_STYLE_BRACKET)
-    local name, description = GetAchievementInfo(achId)
     local catName = "/"
 
     if categoryId ~= false then
@@ -61,14 +60,14 @@ function AchievementInfo.onAchievementUpdated(eventCode, achId)
 
     local numCriteria = GetAchievementNumCriteria(achId)
     for i = 1, numCriteria, 1 do
-        local name, numCompleted, numRequired = GetAchievementCriterion(achId, i)
+        local description, numCompleted, numRequired = GetAchievementCriterion(achId, i)
         local tmpOutput = ""
 
         if i > 1 and AchievementInfo.settingGet("genOnePerLine") == false then
             tmpOutput = tmpOutput .. ", "
         end
 
-        tmpOutput = tmpOutput .. name .. " "
+        tmpOutput = tmpOutput .. description .. " "
         tmpOutput = tmpOutput .. AchievementInfo.calcCriteriaColor(numCompleted, numRequired) .. numCompleted .. "|r"
         tmpOutput = tmpOutput .. AchievementInfo.clrDefault .. "/" .. "|r"
         tmpOutput = tmpOutput .. AchievementInfo.clrCriteriaComplete .. numRequired .. "|r"
@@ -163,13 +162,11 @@ end
 -- Get the correct achievement category
 function AchievementInfo.getCorrectAchievementCategoryId(achId)
     local previousAchievementId = GetPreviousAchievementInLine(achId)
-    local categoryId            = 0
 
     if AchievementInfo.checkForValidCategory(achId) == false and previousAchievementId ~= 0 then
         return AchievementInfo.getCorrectAchievementCategoryId(previousAchievementId)
     elseif AchievementInfo.checkForValidCategory(achId) then
-        categoryId = GetCategoryInfoFromAchievementId(achId)
+        return GetCategoryInfoFromAchievementId(achId)
-        return categoryId
     else
         return false
     end

AchievementInfoSettings.lua

@@ -122,10 +122,9 @@ function AchievementInfo.createSettingsPanel()
 
     -- Add categories dynamically
     local numCats = GetNumAchievementCategories()
-    local catCount = 1
 
     for i = 1, numCats, 1 do
-        catName, numSubCats = GetAchievementCategoryInfo(i)
+        catName = GetAchievementCategoryInfo(i)
 
         table.insert(optionsTable, {
             type = "checkbox",