Skip to content

Don't ignore .e2e/ directories in gitignore template#52663

Merged
baronfel merged 1 commit intodotnet:mainfrom
MattKotsenas:bugfix/e2e-gitignore
Jan 29, 2026
Merged

Don't ignore .e2e/ directories in gitignore template#52663
baronfel merged 1 commit intodotnet:mainfrom
MattKotsenas:bugfix/e2e-gitignore

Conversation

@MattKotsenas
Copy link
Member

Add a negation to avoid gitignore-ing *.e2e/ directories, as that's a common suffix for end-to-end test frameworks. Here's some in-the-wild examples today that would be impacted without this fix: https://github.com/search?q=path%3A.e2e%2F&type=code

@MattKotsenas MattKotsenas requested a review from a team as a code owner January 23, 2026 20:22
Copilot AI review requested due to automatic review settings January 23, 2026 20:22
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

Updates the dotnet new gitignore template to avoid ignoring directories that end in .e2e (commonly used for end-to-end test folders), while still ignoring .e2e trace files.

Changes:

  • Add a negated ignore rule to re-include *.e2e/ directories after the existing *.e2e ignore pattern.
  • Update the integration test approval snapshot to match the new template output.

Reviewed changes

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

File Description
template_feed/Microsoft.DotNet.Common.ItemTemplates/content/Gitignore/.gitignore Adds !*.e2e/ to ensure .e2e directories aren’t ignored by the template.
test/dotnet-new.IntegrationTests/Approvals/AllCommonItemsCreate.-o#dotnet-gitignore-file#-n#item.verified/dotnet-gitignore-file/.gitignore Updates the approved output to reflect the template change.

@baronfel
Copy link
Member

/ba-g templating known issues

@baronfel baronfel enabled auto-merge (squash) January 29, 2026 18:01
@baronfel baronfel merged commit e9af024 into dotnet:main Jan 29, 2026
29 of 33 checks passed
MichaelSimons pushed a commit to MichaelSimons/sdk that referenced this pull request Jan 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants