Extract tables to functions: missing return statement

[kkharji]

SOO finally, I got to try this. Oh it made my life so much easier and with null-ls is crazy good.

Great work 😍 I can only imagine the possibilities of such a codebase.

I notice while extracting tables that it misses adding return keyword.

local function mysources(builtins, dig, fmt)
                {
                      fmt.nixfmt,
                      builtins.code_actions.refactoring,
                      fmt.fixjson,
                      fmt.clang_format,
                      fmt.stylua.with {
                        extra_args = {
                          "--config-path",
                          vim.env.HOME .. "/.config/nvim/stylua.toml",
                          "-",
                        },
                      },
                      dig.vint,
                      fmt.swiftformat.with {
                        extra_args = {
                          "--config",
                          ".swiftformat",
                        },
                      },
                    }
end

Also why is it indent like that 🤔

[teddylear]

@tami5 thanks for opening issue! Do you mind sharing what the code looked like prior to the refactor so I can setup a test case to reproduce the issue? Also is the lua? I'm assuming this is from 'local function'. Indenting for all refactoring operations is something new we have added, so we are still trying to get it right.

[kkharji]

@tami5 thanks for opening issue! Do you mind sharing what the code looked like prior to the refactor so I can setup a test case to reproduce the issue? Also is the lua? I'm assuming this is from 'local function'. Indenting for all refactoring operations is something new we have added, so we are still trying to get it right.

No at all it somewhere in my dot files I don't have access to right now, but basically it visual select a table with in a table and extract to function.

Yes that lua.

Indention maybe could be set by the user?