refactor: make toggle functionality more concise

vim.b.tidy_enabled appears to be redundant
because of M.enabled. This commit removes it.
main
McAuley Penney 1 year ago
parent b157a2b511
commit ae4f9b5e82

@ -3,16 +3,12 @@ local M = {}
M.enabled = true M.enabled = true
function M.toggle() function M.toggle()
if vim.b.tidy_enabled == false then M.enabled = not M.enabled
vim.b.tidy_enabled = nil
M.enabled = true if not M.enabled then
vim.notify("Tidy disabled on save", vim.log.levels.WARN, { title = "Tidy" })
else else
M.enabled = not M.enabled
end
if M.enabled then
vim.notify("Tidy enabled on save", vim.log.levels.INFO, { title = "Tidy" }) vim.notify("Tidy enabled on save", vim.log.levels.INFO, { title = "Tidy" })
else
vim.notify("Tidy disabled on save", vim.log.levels.WARN, { title = "Tidy" })
end end
end end
@ -61,10 +57,10 @@ function M.setup(opts)
local tidy_grp = vim.api.nvim_create_augroup("tidy", { clear = true }) local tidy_grp = vim.api.nvim_create_augroup("tidy", { clear = true })
vim.api.nvim_create_autocmd("BufWritePre", { vim.api.nvim_create_autocmd("BufWritePre", {
group = tidy_grp, group = tidy_grp,
callback = function() callback = function()
if not M.enabled or vim.b.tidy_enabled == false or is_excluded_ft(opts) then if not M.enabled or is_excluded_ft(opts) then
return false return false
end end

Loading…
Cancel
Save