Skip to content

remove .vscode and gitignore it#4498

Merged
jayconrod merged 2 commits intobazel-contrib:masterfrom
jmhodges:remove-vscode-but-document-settings
Nov 4, 2025
Merged

remove .vscode and gitignore it#4498
jayconrod merged 2 commits intobazel-contrib:masterfrom
jmhodges:remove-vscode-but-document-settings

Conversation

@jmhodges
Copy link
Copy Markdown
Contributor

@jmhodges jmhodges commented Nov 2, 2025

What type of PR is this?

Other

What does this PR do? Why is it needed?

The .vscode directory isn't meant to be checked in like this, and causes
churn that vscode users have to remove before sending PRs. We remove it,
add it to .gitignore so future folks don't accidentally check it in.
Folks can check out the more general Editor setup wiki page for
instructions.

Which issues(s) does this PR fix?

Fixes #4497.

Copy link
Copy Markdown
Member

@fmeum fmeum left a comment

Choose a reason for hiding this comment

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

@jmhodges jmhodges force-pushed the remove-vscode-but-document-settings branch from a31e2d2 to 9569e0f Compare November 3, 2025 22:28
@jmhodges jmhodges changed the title remove .vscode and document its settings remove .vscode and gitignore it Nov 4, 2025
The .vscode directory isn't meant to be checked in like this, and causes
churn that vscode users have to remove before sending PRs. We remove it,
add it to .gitignore so future folks don't accidentally check it in.
Folks can check out the more general Editor setup wiki page for
instructions.

Fixes bazel-contrib#4497.
@jmhodges jmhodges force-pushed the remove-vscode-but-document-settings branch from 9569e0f to abd23a4 Compare November 4, 2025 01:21
@jayconrod jayconrod enabled auto-merge (squash) November 4, 2025 15:18
@jayconrod jayconrod merged commit cdf4651 into bazel-contrib:master Nov 4, 2025
1 check passed
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.

checked in .vscode directory causing problems during local dev

3 participants