-
Notifications
You must be signed in to change notification settings - Fork 29
add option to add call_arg_parenthesis
when they are missing
#179
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
What plugin do you currently use to access EmmyLuaCodeStyle formatting |
I've added the option |
I currently use the CLI (latest release https://github.com/CppCXY/EmmyLuaCodeStyle/tree/1.5.4) as well as lua-ls. Using a CLI built from source (latest |
lua-ls is not used through CLI; it is compiled from source and used as a lua-lib. |
I expected that much, though it seems as if they even vendor it, as I was unable to find a replaceable reference to this project in my package management. I checked, they do, I am happy to try to patch it locally. Can you please confirm that the latest |
I don't branch unless it's an important update |
I have been updating my own language server, so I have been somewhat neglecting updates for this project. If you are interested, you can follow my own language server. |
Thanks for pointing that out, though I heavily rely on autogenerated Maybe if your alternative implementation would support loading a |
We implement according to the same standards. We will discuss the implementation of many features, but he(sumneko) doesn't have time to update his ls, while I will implement these features. However, I do not directly support reading .luarc.json. Initially, that's what I did, but I found that many settings were unnecessary. Usually, directly copying the .luarc.json configuration to my configuration works just fine. |
currently
call_arg_parentheses
seems only to be able to remove parenthesis when they are there. Though it can not add them if they are missing.As I consider constructs like
require"ext".setup{}
confusing, I'd prefer if emmy would normalize to always have parenthesis.The text was updated successfully, but these errors were encountered: