Skip to content

Update .gitignore to ignore all .exe files #123220

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
stephentoub merged 2 commits into from
Jan 20, 2026
Merged

Update .gitignore to ignore all .exe files #123220

stephentoub merged 2 commits into from
Jan 20, 2026

Conversation

Copy link
Contributor

Copilot AI commented Jan 15, 2026 •
edited
Loading

Description

Adds *.exe to .gitignore to prevent accidental commits of executable files like nuget.exe.

Changes:

  • Added *.exe pattern to ignore all executable files
  • Added exceptions for existing test resource .exe files that must remain tracked:
    • src/libraries/System.Diagnostics.FileVersionInfo/tests/.../NativeConsoleApp.exe
    • src/libraries/System.Reflection.Metadata/tests/Resources/NetModule/AppCS.exe
    • src/libraries/System.Reflection.Metadata/tests/Resources/Misc/Signed.exe
Original prompt

copilot has an unfortunate habit of trying to merge nuget.exe into the repo. Please ensure that the .gitignore file ignores all .exe files.

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@stephentoub
Update .gitignore to ignore all .exe files
dd5e08f 
Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>
Copilot AI changed the title [WIP] Update .gitignore to exclude all .exe files Update .gitignore to ignore all .exe files Jan 15, 2026
Copilot AI requested a review from stephentoub January 15, 2026 16:22
@stephentoub stephentoub marked this pull request as ready for review January 15, 2026 16:31
Copilot AI review requested due to automatic review settings January 15, 2026 16:31
Copilot AI reviewed Jan 15, 2026
View reviewed changes
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.

Copilot wasn't able to review any files in this pull request.

@stephentoub stephentoub requested a review from ericstj January 15, 2026 16:31
@stephentoub stephentoub enabled auto-merge (squash) January 15, 2026 16:32
@dotnet-policy-service
Copy link
Contributor

dotnet-policy-service bot commented Jan 15, 2026

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

@build-analysis build-analysis bot mentioned this pull request Jan 15, 2026
Open
@stephentoub stephentoub requested a review from jkotas January 20, 2026 03:09
@stephentoub stephentoub merged commit facd688 into main Jan 20, 2026
156 of 159 checks passed
@jkotas jkotas deleted the copilot/update-gitignore-to-ignore-exe branch January 20, 2026 03:13
@dotnet-maestro dotnet-maestro bot mentioned this pull request Jan 21, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@jkotas jkotas jkotas approved these changes

@ericstj ericstj Awaiting requested review from ericstj

+1 more reviewer

@stephentoub stephentoub stephentoub approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@stephentoub stephentoub

Copilot code review Copilot

Labels

area-Infrastructure

Projects

Runtime Infra
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@stephentoub @jkotas