Skip to content

Updated gitignore #85

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

Closed
wants to merge 1 commit into from
Closed

Updated gitignore #85

wants to merge 1 commit into from

Conversation

ProfessionalMenace
Copy link
Contributor

removed poetry.lock (should be created automatically) added poetry.lock to .gitignore
removed unused stuff from .gitignore

removed poetry.lock (should be created automatically)
added poetry.lock to .gitignore
removed unused stuff from .gitignore
@vesper-arch
Copy link
Owner

Some of these I do have to use. pyright settings I have as well as the virtual environment directories. Probably some other stuff I'll have to check later

@vesper-arch
Copy link
Owner

Unit test stuff also is needed I think

Copy link
Collaborator

@miketwo miketwo left a comment

Choose a reason for hiding this comment

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

What problem is this supposed to fix?

# C extensions
*.so
# ignore poetry lock file - created automatically
poetry.lock
Copy link
Collaborator

Choose a reason for hiding this comment

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

Poetry recommends committing the lockfile to get more reproducible builds. It makes it so that when dependencies are installed, the exact ones are used for different developers.

Comment on lines -39 to -54
# Unit test / coverage reports
htmlcov/
.tox/
.nox/
.coverage
.coverage.*
.cache
nosetests.xml
coverage.xml
*.cover
*.py,cover
.hypothesis/
.pytest_cache/
cover/
pytest-coverage.txt
pytest.xml
Copy link
Collaborator

Choose a reason for hiding this comment

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

All of these are good to have because the repo has unit tests

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