8000 Make Lute a single-folder VS Code workspace by vrn-sn · Pull Request #279 · luau-lang/lute · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Make Lute a single-folder VS Code workspace #279

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 13, 2025

Conversation

vrn-sn
Copy link
Member
@vrn-sn vrn-sn commented May 13, 2025

VS Code has varying levels of settings overrides. The most basic is user settings, then workspace settings, followed by project folder settings. Currently, we store project settings in .vscode/settings.json, which means that we can't add the .vscode directory to .gitignore.

This PR creates a multi-root workspace containing only the root folder of this repository, which allows us to store these settings in lute.code-workspace instead. Now, .vscode is added to .gitignore and can be used for defining local-only launch and task configurations.

This also means that we can add workspace-level launch and task configurations (which would run luthier) that come packaged with this repository by default in lute.code-workspace.

As long as the repository is opened in VS Code as a workspace, this should behave the same as before.

@vrn-sn vrn-sn requested a review from aatxe May 13, 2025 23:52
Copy link
Collaborator
@aatxe aatxe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I still don't use VS Code, but this looks broadly more principled to me, so I'm happy to do it this way.

@aatxe aatxe merged commit 3626d66 into primary May 13, 2025
6 checks passed
@aatxe aatxe deleted the vrn-sn/move-settings-json-to-code-workspace branch May 13, 2025 23:55
green-real pushed a commit to green-real/lute that referenced this pull request May 19, 2025
VS Code has varying levels of settings overrides. The most basic is user
settings, then workspace settings, followed by project folder settings.
Currently, we store project settings in `.vscode/settings.json`, which
means that we can't add the `.vscode` directory to `.gitignore`.

This PR creates a [multi-root
workspace](https://code.visualstudio.com/docs/editing/workspaces/multi-root-workspaces)
containing only the root folder of this repository, which allows us to
store these settings in `lute.code-workspace` instead. Now, `.vscode` is
added to `.gitignore` and can be used for defining local-only launch and
task configurations.

This also means that we can add workspace-level launch and task
configurations (which would run luthier) that come packaged with this
repository by default in `lute.code-workspace`.

As long as the repository is opened in VS Code as a workspace, this
should behave the same as before.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0