Skip to content

Use VSCode-EmmyLua config to provide global config #22

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

Merged
merged 1 commit into from
May 27, 2025

Conversation

georgiy-belyanin
Copy link
Member

In e356b99 ('Set up language server configuration automatically') logic
for setting up EmmyLuaLs automatically has been added. It used
a configuration file at the home directory.

It's not really very cool to spam with extension-specific files at the
user's home dir. A patch [^1] has been introduced allowing one to
specify a path to a global language server configuration file within the
VS Code options. Let's set up this instead.

[^1] EmmyLua/VSCode-EmmyLua#227

@georgiy-belyanin georgiy-belyanin force-pushed the conf-emmylua-via-vscode-opt branch from f79afc5 to 716e7a2 Compare May 27, 2025 08:58
In e356b99 ('Set up language server configuration automatically') logic
for setting up EmmyLuaLs automatically has been added. It used
a configuration file at the home directory.

It's not really very cool to spam with extension-specific files at the
user's home dir. A patch [^1] has been introduced allowing one to
specify a path to a global language server configuration file within the
VS Code options. Let's set up this instead.

[^1] EmmyLua/VSCode-EmmyLua#227
@georgiy-belyanin georgiy-belyanin force-pushed the conf-emmylua-via-vscode-opt branch from 716e7a2 to 757c6a3 Compare May 27, 2025 09:03
@georgiy-belyanin georgiy-belyanin merged commit 048c996 into main May 27, 2025
3 checks passed
@georgiy-belyanin georgiy-belyanin deleted the conf-emmylua-via-vscode-opt branch May 27, 2025 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant