build: fix stylua check (#409)

Author: MunifTanjim

.github/workflows/ci.yml

@@ -19,7 +19,7 @@ jobs:
       uses: JohnnyMorganz/[email protected]
       with:
         token: ${{ secrets.GITHUB_TOKEN }}
-        args: --glob '*.lua' --glob '!defaults.lua' --check lua 
+        args: --color always --check lua/
 
   plenary-tests:
     runs-on: ubuntu-20.04

lua/neo-tree.lua

@@ -32,7 +32,6 @@ local get_position = function(source_name)
   return pos
 end
 
-
 M.ensure_config = function()
   if not M.config then
     M.setup({ log_to_file = false }, true)
@@ -243,7 +242,6 @@ M.setup = function(config, is_auto_config)
   if not is_auto_config and netrw.get_hijack_netrw_behavior() ~= "disabled" then
     vim.cmd("silent! autocmd! FileExplorer *")
   end
-
 end
 
 M.show_logs = function()

lua/neo-tree/events/queue.lua

@@ -3,7 +3,7 @@ local log = require("neo-tree.log")
 
 Node = {}
 function Node:new(value)
-  local props = {prev = nil, next = nil, value = value}
+  local props = { prev = nil, next = nil, value = value }
   setmetatable(props, self)
   self.__index = self
   return props

lua/neo-tree/git/status.lua

@@ -144,7 +144,7 @@ M.status = function(base, exclude_directories, path)
     git_root = git_root,
     git_status = {},
     exclude_directories = exclude_directories,
-    lines_parsed = 0
+    lines_parsed = 0,
   }
 
   for _, line in ipairs(staged_result) do
@@ -232,7 +232,7 @@ M.status_async = function(path, base, opts)
     return true
   end
 
-  local job_complete_callback = function ()
+  local job_complete_callback = function()
     utils.debounce(event_id, nil, nil, nil, utils.debounce_action.COMPLETE_ASYNC_JOB)
     vim.schedule(function()
       events.fire_event(events.GIT_STATUS_CHANGED, {
@@ -247,67 +247,65 @@ M.status_async = function(path, base, opts)
   end)
 
   utils.debounce(event_id, function()
-    local staged_job = Job
-      :new({
-        command = "git",
-        args = { "-C", git_root, "diff", "--staged", "--name-status", base, "--" },
-        enable_recording = false,
-        maximium_results = context.max_lines,
-        on_stdout = vim.schedule_wrap(function(err, line, job)
-          if should_process(err, line, job, "status_async staged error:") then
-            table.insert(context.lines, line)
-          end
-        end),
-        on_stderr = function(err, line)
-          if err and err > 0 then
-            log.error("status_async staged error: ", err, line)
-          end
-        end,
-      })
+    local staged_job = Job:new({
+      command = "git",
+      args = { "-C", git_root, "diff", "--staged", "--name-status", base, "--" },
+      enable_recording = false,
+      maximium_results = context.max_lines,
+      on_stdout = vim.schedule_wrap(function(err, line, job)
+        if should_process(err, line, job, "status_async staged error:") then
+          table.insert(context.lines, line)
+        end
+      end),
+      on_stderr = function(err, line)
+        if err and err > 0 then
+          log.error("status_async staged error: ", err, line)
+        end
+      end,
+    })
 
-    local unstaged_job = Job
-      :new({
-        command = "git",
-        args = { "-C", git_root, "diff", "--name-status" },
-        enable_recording = false,
-        maximium_results = context.max_lines,
-        on_stdout = vim.schedule_wrap(function(err, line, job)
-          if should_process(err, line, job, "status_async unstaged error:") then
-            if line then
-              line = " " .. line
-            end
-            table.insert(context.lines, line)
+    local unstaged_job = Job:new({
+      command = "git",
+      args = { "-C", git_root, "diff", "--name-status" },
+      enable_recording = false,
+      maximium_results = context.max_lines,
+      on_stdout = vim.schedule_wrap(function(err, line, job)
+        if should_process(err, line, job, "status_async unstaged error:") then
+          if line then
+            line = " " .. line
           end
-        end),
-        on_stderr = function(err, line)
-          if err and err > 0 then
-            log.error("status_async unstaged error: ", err, line)
-          end
-        end,
-      })
+          table.insert(context.lines, line)
+        end
+      end),
+      on_stderr = function(err, line)
+        if err and err > 0 then
+          log.error("status_async unstaged error: ", err, line)
+        end
+      end,
+    })
 
-    local untracked_job = Job
-      :new({
-        command = "git",
-        args = { "-C", git_root, "ls-files", "--exclude-standard", "--others" },
-        enable_recording = false,
-        maximium_results = context.max_lines,
-        on_stdout = vim.schedule_wrap(function(err, line, job)
-          if should_process(err, line, job, "status_async untracked error:") then
-            if line then
-              line = "?	" .. line
-            end
-            table.insert(context.lines, line)
+    local untracked_job = Job:new({
+      command = "git",
+      args = { "-C", git_root, "ls-files", "--exclude-standard", "--others" },
+      enable_recording = false,
+      maximium_results = context.max_lines,
+      on_stdout = vim.schedule_wrap(function(err, line, job)
+        if should_process(err, line, job, "status_async untracked error:") then
+          if line then
+            line = "?	" .. line
           end
-        end),
-        on_stderr = function(err, line)
-          if err and err > 0 then
-            log.error("status_async untracked error: ", err, line)
-          end
-        end,
-      })
+          table.insert(context.lines, line)
+        end
+      end),
+      on_stderr = function(err, line)
+        if err and err > 0 then
+          log.error("status_async untracked error: ", err, line)
+        end
+      end,
+    })
 
-    local showUntracked = vim.fn.systemlist({"git", "-C", git_root, "config", "--get", "status.showUntrackedFiles"})
+    local showUntracked =
+      vim.fn.systemlist({ "git", "-C", git_root, "config", "--get", "status.showUntrackedFiles" })
     log.debug("git status.showUntrackedFiles =", showUntracked[1])
     if showUntracked[1] == "no" then
       unstaged_job:after(parse_lines)

lua/neo-tree/log.lua

@@ -6,7 +6,6 @@
 -- This library is free software; you can redistribute it and/or modify it
 -- under the terms of the MIT license. See LICENSE for details.
 
-
 local vim = vim
 -- User configuration section
 local default_config = {
@@ -28,8 +27,8 @@ local default_config = {
   -- Level configuration
   modes = {
     { name = "trace", hl = "None", level = vim.log.levels.TRACE },
-    { name = "debug", hl = "None" , level = vim.log.levels.DEBGUG},
-    { name = "info", hl = "None" , level = vim.log.levels.INFO},
+    { name = "debug", hl = "None", level = vim.log.levels.DEBGUG },
+    { name = "info", hl = "None", level = vim.log.levels.INFO },
     { name = "warn", hl = "WarningMsg", level = vim.log.levels.WARN },
     { name = "error", hl = "ErrorMsg", level = vim.log.levels.ERROR },
     { name = "fatal", hl = "ErrorMsg", level = vim.log.levels.ERROR },
@@ -44,25 +43,22 @@ local log = {}
 
 local unpack = unpack or table.unpack
 
-local notify = function (message, level_config)
+local notify = function(message, level_config)
   if type(vim.notify) == "table" then
     -- probably using nvim-notify
     vim.notify(message, level_config.level, { title = "Neo-tree" })
   else
     local nameupper = level_config.name:upper()
-    local console_string = string.format( "[Neo-tree %s] %s", nameupper, message)
+    local console_string = string.format("[Neo-tree %s] %s", nameupper, message)
     vim.notify(console_string, level_config.level)
   end
 end
 
 log.new = function(config, standalone)
   config = vim.tbl_deep_extend("force", default_config, config)
 
-  local outfile = string.format(
-    "%s/%s.log",
-    vim.api.nvim_call_function("stdpath", { "data" }),
-    config.plugin
-  )
+  local outfile =
+    string.format("%s/%s.log", vim.api.nvim_call_function("stdpath", { "data" }), config.plugin)
 
   local obj
   if standalone then
@@ -96,7 +92,6 @@ log.new = function(config, standalone)
     levels[v.name] = i
   end
 
-
   obj.set_level = function(level)
     if levels[level] then
       if config.level ~= level then

lua/neo-tree/setup/deprecations.lua

@@ -1,4 +1,4 @@
-local utils = require "neo-tree.utils"
+local utils = require("neo-tree.utils")
 
 local M = {}
 
@@ -9,7 +9,11 @@ M.show_migrations = function()
     for i, message in ipairs(migrations) do
       migrations[i] = "  * " .. message
     end
-    table.insert(migrations, 1, "# Neo-tree configuration has been updated. Please review the changes below.")
+    table.insert(
+      migrations,
+      1,
+      "# Neo-tree configuration has been updated. Please review the changes below."
+    )
     local buf = vim.api.nvim_create_buf(false, true)
     vim.api.nvim_buf_set_lines(buf, 0, -1, false, migrations)
     vim.api.nvim_buf_set_option(buf, "buftype", "nofile")
@@ -19,25 +23,26 @@ M.show_migrations = function()
     vim.api.nvim_buf_set_option(buf, "modifiable", false)
     vim.api.nvim_buf_set_option(buf, "filetype", "markdown")
     vim.api.nvim_buf_set_name(buf, "Neo-tree migrations")
-    vim.defer_fn(function ()
+    vim.defer_fn(function()
       vim.cmd(string.format("%ssplit", #migrations))
       vim.api.nvim_win_set_buf(0, buf)
     end, 100)
   end
 end
 
-M.migrate = function (config)
+M.migrate = function(config)
   migrations = {}
 
-  local moved = function (old, new, converter)
+  local moved = function(old, new, converter)
     local exising = utils.get_value(config, old)
     if type(exising) ~= "nil" then
       if type(converter) == "function" then
         exising = converter(exising)
       end
       utils.set_value(config, new, exising)
       config[old] = nil
-      migrations[#migrations + 1] = string.format("The `%s` option has been deprecated, please use `%s` instead.", old, new)
+      migrations[#migrations + 1] =
+        string.format("The `%s` option has been deprecated, please use `%s` instead.", old, new)
     end
   end
 
@@ -53,11 +58,12 @@ M.migrate = function (config)
     local value = utils.get_value(config, key)
     if value == old_value then
       utils.set_value(config, key, new_value)
-      migrations[#migrations + 1] = string.format("The `%s=%s` option has been renamed to `%s`.", key, old_value, new_value)
+      migrations[#migrations + 1] =
+        string.format("The `%s=%s` option has been renamed to `%s`.", key, old_value, new_value)
     end
   end
 
-  local opposite = function (value)
+  local opposite = function(value)
     return not value
   end
 
@@ -67,7 +73,7 @@ M.migrate = function (config)
   removed("filesystem.filters.gitignore_source")
   removed("filesystem.filter_items.gitignore_source")
   renamed_value("filesystem.hijack_netrw_behavior", "open_split", "open_current")
-  for _, source in ipairs({"filesystem", "buffers", "git_status"}) do
+  for _, source in ipairs({ "filesystem", "buffers", "git_status" }) do
     renamed_value(source .. "window.position", "split", "current")
   end
 

lua/neo-tree/setup/init.lua

@@ -15,7 +15,7 @@ local sources = {
   "filesystem",
   "buffers",
   "git_status",
-  "example"
+  "example",
 }
 
 local M = {}
@@ -50,22 +50,23 @@ local define_events = function()
     return args
   end)
 
-  events.define_autocmd_event(
-    events.VIM_BUFFER_CHANGED,
-    { "BufWritePost", "BufFilePost" },
-    200
-  )
+  events.define_autocmd_event(events.VIM_BUFFER_CHANGED, { "BufWritePost", "BufFilePost" }, 200)
 
-  events.define_autocmd_event(events.VIM_BUFFER_MODIFIED_SET, { "BufModifiedSet" }, 0, function(args)
-    if utils.is_real_file(args.afile) then
-      -- we could use args.afile to update the sigle file that changed, but it seems like we miss
-      -- buffers when `:wa` is used.
-      args.modified_buffers = utils.get_modified_buffers()
-      return args
-    else
-      return false
+  events.define_autocmd_event(
+    events.VIM_BUFFER_MODIFIED_SET,
+    { "BufModifiedSet" },
+    0,
+    function(args)
+      if utils.is_real_file(args.afile) then
+        -- we could use args.afile to update the sigle file that changed, but it seems like we miss
+        -- buffers when `:wa` is used.
+        args.modified_buffers = utils.get_modified_buffers()
+        return args
+      else
+        return false
+      end
     end
-  end)
+  )
 
   events.define_autocmd_event(events.VIM_BUFFER_ADDED, { "BufAdd" }, 200)
   events.define_autocmd_event(events.VIM_BUFFER_DELETED, { "BufDelete" }, 200)
@@ -78,15 +79,15 @@ local define_events = function()
   events.define_autocmd_event(events.VIM_LEAVE, { "VimLeavePre" })
   events.define_autocmd_event(events.VIM_WIN_CLOSED, { "WinClosed" })
   events.define_autocmd_event(events.VIM_COLORSCHEME, { "ColorScheme" }, 0)
-  events.define_autocmd_event(events.GIT_EVENT, { "User FugitiveChanged" }, 100 )
+  events.define_autocmd_event(events.GIT_EVENT, { "User FugitiveChanged" }, 100)
   events.define_event(events.GIT_STATUS_CHANGED, { debounce_frequency = 0 })
   events_setup = true
 
   events.subscribe({
     event = events.VIM_LEAVE,
     handler = function()
       events.clear_all_events()
-    end
+    end,
   })
 end
 
@@ -100,9 +101,10 @@ M.buffer_enter_event = function()
     setlocal nolist nospell nonumber norelativenumber
     ]])
 
-    local winhighlight = 'Normal:NeoTreeNormal,NormalNC:NeoTreeNormalNC,SignColumn:NeoTreeSignColumn,CursorLine:NeoTreeCursorLine,FloatBorder:NeoTreeFloatBorder,StatusLine:NeoTreeStatusLine,StatusLineNC:NeoTreeStatusLineNC,VertSplit:NeoTreeVertSplit,EndOfBuffer:NeoTreeEndOfBuffer'
+    local winhighlight =
+      "Normal:NeoTreeNormal,NormalNC:NeoTreeNormalNC,SignColumn:NeoTreeSignColumn,CursorLine:NeoTreeCursorLine,FloatBorder:NeoTreeFloatBorder,StatusLine:NeoTreeStatusLine,StatusLineNC:NeoTreeStatusLineNC,VertSplit:NeoTreeVertSplit,EndOfBuffer:NeoTreeEndOfBuffer"
     if vim.version().minor >= 7 then
-      vim.cmd("setlocal winhighlight=" .. winhighlight .. ',WinSeparator:NeoTreeWinSeparator')
+      vim.cmd("setlocal winhighlight=" .. winhighlight .. ",WinSeparator:NeoTreeWinSeparator")
     else
       vim.cmd("setlocal winhighlight=" .. winhighlight)
     end
@@ -409,10 +411,11 @@ M.merge_config = function(user_config, is_auto_config)
   normalize_mappings(user_config)
   merge_renderers(default_config, nil, user_config)
   for _, source_name in ipairs(sources) do
-    default_config[source_name] = default_config[source_name] or {
-      renderers = {},
-      components = {},
-    }
+    default_config[source_name] = default_config[source_name]
+      or {
+        renderers = {},
+        components = {},
+      }
     local source_default_config = default_config[source_name]
     local mod_root = "neo-tree.sources." .. source_name
     source_default_config.components = require(mod_root .. ".components")