Fix .vscodeignore missing .gitignore in themes extension - #14636
Conversation
There was a problem hiding this comment.
🟡 Not ready to approve
Themes/.npmrc exists but is not excluded, so development registry/auth configuration would be packaged into the published extension.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.
Pull request overview
Updates the Themes extension packaging ignore list to avoid shipping development-only files in the published VS Code extension.
Changes:
- Replaces the package-lock-only ignore block with a broader “development files” block.
- Adds
.gitignoreto the Themes extension.vscodeignore.
File summaries
| File | Description |
|---|---|
| Themes/.vscodeignore | Expands the ignore list for the Themes extension package to exclude additional development files. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 1
- Review effort level: Lite
We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🟢 Ready to approve
The change is small, consistent with VS Code extension packaging behavior, and introduces no functional/runtime code impact.
This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0 new
- Review effort level: Lite
We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.
No description provided.