Skip to content

Add .vscode/ to .gitignore #85

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Sep 4, 2025
Merged

Add .vscode/ to .gitignore #85

merged 2 commits into from
Sep 4, 2025

Conversation

raftaar1191
Copy link
Contributor

Description

This PR adds .vscode/ to the .gitignore file to prevent VS Code workspace settings from being committed to the repository.

Changes Made

  • Added .vscode/ entry under the "AI Tools and IDEs" section in .gitignore
  • Updated section header from "AI Tools" to "AI Tools and IDEs" for better organization

Why This Change Is Needed

  • Prevents conflicts: VS Code workspace settings can override other developers' IDE preferences
  • Avoids accidents: Developers won't accidentally commit personal/local IDE configuration
  • Best practices: Following common conventions for PHP projects
  • Developer freedom: Allows team members to customize their local VS Code setup without affecting the repository

Resolves

Closes #84

Testing

  • Verified that .vscode/ directory and files are now ignored by git
  • No impact on existing functionality

Screenshots/Examples

Before:

.claude/
CLAUDE.md
.clinerules/
.cursor/
GEMINI.md

After:

.claude/
CLAUDE.md
.clinerules/
.cursor/
.vscode/
GEMINI.md

@raftaar1191
Add .vscode/ to .gitignore
0a9f247 
- Prevents VS Code workspace settings from being committed
- Allows developers to customize local IDE configuration
- Follows common best practices for PHP projects
- Resolves issue WordPress#84
@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 3, 2025 •
edited
Loading

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: raftaar1191 <[email protected]>
Co-authored-by: JasonTheAdams <[email protected]>
Co-authored-by: swissspidy <[email protected]>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

JasonTheAdams
JasonTheAdams requested changes Sep 4, 2025
View reviewed changes
Copy link
Member

@JasonTheAdams JasonTheAdams left a comment

Choose a reason for hiding this comment

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

Thanks! I actually just did this in another PR. Would you mind changing it to use this format: https://github.com/WordPress/php-ai-client/blob/5ea1fefba167bee2bd68f62020a3c555a97f8fe2/.gitignore

@raftaar1191
Copy link
Contributor Author

Done @JasonTheAdams

Update the code as per: https://github.com/WordPress/php-ai-client/blob/5ea1fefba167bee2bd68f62020a3c555a97f8fe2/.gitignore

JasonTheAdams
JasonTheAdams approved these changes Sep 4, 2025
View reviewed changes
Copy link
Member

@JasonTheAdams JasonTheAdams left a comment

Choose a reason for hiding this comment

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

Thank you! Appreciate the contribution! 🙌

@JasonTheAdams JasonTheAdams merged commit 20e9b9d into WordPress:trunk Sep 4, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JasonTheAdams JasonTheAdams JasonTheAdams approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add .vscode/ to .gitignore
2 participants
@raftaar1191 @JasonTheAdams