Skip to content
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.

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

Create .gitignore file #76

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gustoliveira
Copy link

I ran test coverage to this project and notice the coverage.out file was available to commit.

With this in mind, this pull request adds a .gitignore file to the project, which was previously missing.

@lithammer
Copy link
Owner

How did you specifically run test coverage? Given that you can pick any name I'm not sure how relevant a .gitignore file with *.out is?

$ go test -coverprofile=coverage
$ go tool cover -html=coverage

I'm also not a fan of generic ignore files that contains a bunch of entries not even relevant for the project. For example this is a library, so *.exe will never be relevant.

Comment on lines +1 to +21
### Go ###

# Binaries for programs and plugins
*.exe
*.exe~
*.dll
*.so
*.dylib

# Test binary, built with `go test -c`
*.test

# Output of the go coverage tool, specifically when used with LiteIDE
*.out

# Dependency directories (remove the comment below to include it)
# vendor/

# Go workspace file
go.work

Copy link
Owner

@lithammer lithammer Jan 5, 2025

Choose a reason for hiding this comment

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

Remove all the irrelevant entries.

Suggested change
### Go ###
# Binaries for programs and plugins
*.exe
*.exe~
*.dll
*.so
*.dylib
# Test binary, built with `go test -c`
*.test
# Output of the go coverage tool, specifically when used with LiteIDE
*.out
# Dependency directories (remove the comment below to include it)
# vendor/
# Go workspace file
go.work
# Go coverage tool.
/*.out

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.

2 participants