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

style(gitignore): Omit trailing / for consistency #25

Merged
merged 1 commit into from
May 4, 2022
Merged

Conversation

Kurt-von-Laven
Copy link
Contributor

Both report and .venv are directories, and a trailing / indicates that only directories should be matched. However, most directory names, including these, make poor choices of filenames because they lack extensions and would be prone to confusion with the identically named directories. Instructing Git to ignore such files adds a bit of friction to tracking such files.

Both report and .venv are directories, and a trailing / indicates that
only directories should be matched. However, most directory names,
including these, make poor choices of filenames because they lack
extensions and would be prone to confusion with the identically named
directories. Instructing Git to ignore such files adds a bit of friction
to tracking such files.
@Kurt-von-Laven Kurt-von-Laven self-assigned this May 4, 2022
@Kurt-von-Laven Kurt-von-Laven merged commit 3370f1a into main May 4, 2022
@Kurt-von-Laven Kurt-von-Laven deleted the gitignore branch May 4, 2022 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants