Skip to content

Remove vscode folder from repo#1179

Merged
karilint merged 5 commits into
mainfrom
remove-vscode-folder-from-repo
Jun 11, 2026
Merged

Remove vscode folder from repo#1179
karilint merged 5 commits into
mainfrom
remove-vscode-folder-from-repo

Conversation

@juhanikat

Copy link
Copy Markdown
Collaborator

No description provided.

@juhanikat juhanikat force-pushed the remove-vscode-folder-from-repo branch from e08da32 to e100e41 Compare June 9, 2026 10:36
@juhanikat juhanikat requested a review from karilint June 9, 2026 14:02
@karilint karilint requested a review from Copilot June 11, 2026 08:10

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Removes the repository-tracked VS Code workspace settings and ensures the .vscode folder is ignored going forward, keeping editor-specific configuration out of version control.

Changes:

  • Deleted .vscode/settings.json from the repository.
  • Added .vscode/ to .gitignore to prevent re-adding editor settings.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
.vscode/settings.json Removed editor-specific workspace settings from the repo.
.gitignore Ignores .vscode/ to prevent VS Code configuration from being committed again.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@karilint karilint left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Run Copilot review. Approve.

@karilint karilint merged commit 771e372 into main Jun 11, 2026
4 checks passed
@karilint karilint deleted the remove-vscode-folder-from-repo branch June 11, 2026 08:38
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.

3 participants