Skip to content

Remove /** */ auto-closing from gitignore syntax#273553

Merged
kycutler merged 1 commit intomainfrom
kycutler/nogitignorecomments
Oct 27, 2025
Merged

Remove /** */ auto-closing from gitignore syntax#273553
kycutler merged 1 commit intomainfrom
kycutler/nogitignorecomments

Conversation

@kycutler
Copy link
Contributor

Fixes #273372

@kycutler kycutler requested review from Copilot and lszomoru October 27, 2025 13:30
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes the /** */ auto-closing pair from the gitignore language configuration, addressing issue #273372. The change prevents VS Code from automatically inserting */ when typing /** in gitignore files, where this syntax is not valid.

Key changes:

  • Removed the /** */ auto-closing pair configuration from gitignore syntax
  • Updated trailing comma on the previous auto-closing pair entry

@vs-code-engineering vs-code-engineering bot added this to the October 2025 milestone Oct 27, 2025
@TylerLeonhardt TylerLeonhardt added the ~copilot-video-please ⚠️WIP🏗️ Use in PRs. Runs Copilot CLI agent to record a video of "trying out the change" label Oct 27, 2025
@vs-code-engineering vs-code-engineering bot removed the ~copilot-video-please ⚠️WIP🏗️ Use in PRs. Runs Copilot CLI agent to record a video of "trying out the change" label Oct 27, 2025
@vs-code-engineering
Copy link

🤖 Copilot has tried out this change! Members of the team can view the recording here (run)

@kycutler kycutler merged commit b7d8cdd into main Oct 27, 2025
28 checks passed
@kycutler kycutler deleted the kycutler/nogitignorecomments branch October 27, 2025 15:33
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Dec 11, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

/** is automatically closed in languages where that isn't a comment

4 participants