Skip to content
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

feat: Add snippets for VScode #937

Merged
merged 2 commits into from May 14, 2024
Merged

Conversation

c-vetter
Copy link
Contributor

@c-vetter c-vetter commented May 8, 2024

No description provided.

Copy link
Owner

@ploeh ploeh left a comment

Choose a reason for hiding this comment

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

So, let me get this straight... Just by placing the snippets file in the (otherwise Git-ignored) .vscode directory, the code snippets light up for that Git repository?

lotr-lord-of-the-rings

.vscode/contribution.code-snippets Outdated Show resolved Hide resolved
After an update to VScode, the comma-less style with unescaped tabs
was marked invalid. It was considered valid before that ¯\\_(ツ)_/¯
@c-vetter
Copy link
Contributor Author

So, let me get this straight... Just by placing the snippets file in the (otherwise Git-ignored) .vscode directory, the code snippets light up for that Git repository?

That is correct. The .vscode directory is used for repository-local overrides and additions. Another neat feature is that you can recommend extensions for contributors with a .vscode/extensions.json. Although there's no good reason to edit that one manually, because that's what "Add to Workspace Recommendations" is for in the extension settings. Just git add it and it's good to go.

@ploeh ploeh merged commit e68a28d into ploeh:master May 14, 2024
@ploeh
Copy link
Owner

ploeh commented May 19, 2024

FWIW, I'm already enjoying that these snippets are just available in the code base. Thank you 👍

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.

None yet

2 participants