-
Notifications
You must be signed in to change notification settings - Fork 196
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
Update .gitignore
#664
base: main
Are you sure you want to change the base?
Update .gitignore
#664
Conversation
Any particular rationale for this pull request? e.g what's being accidentally included when working on the repository? |
.ropeproject | ||
.idea | ||
# Byte-compiled / optimized / DLL files | ||
__pycache__/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Definitely to be ignored and so it be on top.
_examples/* | ||
docs/* | ||
*.egg | ||
MANIFEST |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May be removed, the project does not use it anyway.
.gitignore
Outdated
# IPython | ||
profile_default/ | ||
ipython_config.py |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This may be removed if you like.
# Sphinx documentation | ||
docs/_build/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I couldn't find what docs generator is preferred, so it may be here or I can remove it.
# Visual Studio Code | ||
.vscode/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added this, but this is tricky and I need you approval for this directory to be ignored:
.gitignore
Outdated
# UV | ||
# Similar to Pipfile.lock, it is generally recommended to include uv.lock in version control. | ||
# This is especially recommended for binary packages to ensure reproducibility, and is more | ||
# commonly ignored for libraries. | ||
#uv.lock |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll remove uv
for now because pipenv
is in use.
# Environments | ||
.env | ||
.venv | ||
*__pycache__* | ||
env/ | ||
venv/ | ||
ENV/ | ||
env.bak/ | ||
venv.bak/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Environments are grouped.
.gitignore
Outdated
# Spyder project settings | ||
.spyderproject | ||
.spyproject | ||
|
||
# Rope project settings | ||
.ropeproject | ||
|
||
# mkdocs documentation | ||
/site |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May be removed.
# Unit test / coverage reports | ||
htmlcov/ | ||
.tox/ | ||
.nox/ | ||
.coverage | ||
.coverage.* | ||
.cache | ||
nosetests.xml | ||
coverage.xml | ||
*.cover | ||
*.py,cover | ||
.hypothesis/ | ||
.pytest_cache/ | ||
cover/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to have them grouped. The entries about nose testing framework may be removed.
Hello. I marked what I saw important of being noted. The rationale is to enrich/refresh the ".gitignore" file. It may be not so advantageous, but we can make it more similar to the more or less commonly adopted Python .gitignore template and without just having it copypasted completely. |
The project ".gitignore" file is partially extended from the Python .gitignore template.