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

Add __pycache__/ to .gitignore #4325

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

Conversation

Avasam
Copy link
Contributor

@Avasam Avasam commented Apr 24, 2024

Summary of changes

Running tox locally leaves a lot of files behind in __pycache__ folders.
image
This PR adds the cache folder to the .gitignore, since this is specific to this project (using tox) and not my personal environment.

Pull Request Checklist

@abravalheri
Copy link
Contributor

https://blog.jaraco.com/skeleton/#ignoring-artifacts does mention __pycache__ in the suggested .gitignore_global, so by that logic, this line would not be necessary...

However there is precedent (the currently ignored *.pyc, .mypy_cache, .pytest_cache, etc...). So I will defer this one to Jason 😅.

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.

None yet

2 participants