Skip to content

Go: Add .gitignore for artifacts of make test#18886

Merged
owen-mc merged 1 commit intogithub:mainfrom
owen-mc:go/gitignore-make-test-artifacts
Feb 27, 2025
Merged

Go: Add .gitignore for artifacts of make test#18886
owen-mc merged 1 commit intogithub:mainfrom
owen-mc:go/gitignore-make-test-artifacts

Conversation

@owen-mc
Copy link
Copy Markdown
Contributor

@owen-mc owen-mc commented Feb 27, 2025

Ignore files which are left behind when you run make test in go.

I ran make test locally with this .gitignore to check that it was doing the right thing. The only artifact that it doesn't block is the stats file, which I think we shouldn't ignore.

Copilot AI review requested due to automatic review settings February 27, 2025 16:36
@owen-mc owen-mc requested a review from a team as a code owner February 27, 2025 16:36
Copy link
Copy Markdown
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 reviewed 1 out of 1 changed files in this pull request and generated no comments.

Tip: If you use Visual Studio Code, you can request a review from Copilot before you push from the "Source Control" tab. Learn more

@owen-mc owen-mc merged commit 93b1114 into github:main Feb 27, 2025
11 checks passed
@owen-mc owen-mc deleted the go/gitignore-make-test-artifacts branch February 27, 2025 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants