8000 Add coverage files to gitignore by Jackenmen · Pull Request #2323 · psf/black · GitHub
[go: up one dir, main page]
Skip to content

Add coverage files to gitignore #2323

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jun 9, 2021
Merged

Add coverage files to gitignore #2323

merged 1 commit into from
Jun 9, 2021

Conversation

Jackenmen
Copy link
Contributor

I've been getting a bunch of these when running tox -e py
image

@ichard26 ichard26 added the skip news Pull requests that don't need a changelog entry. label Jun 9, 2021
felix-hilden
felix-hilden approved these changes Jun 9, 2021
View reviewed changes
Copy link
Collaborator
@felix-hilden felix-hilden left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Now that I run that exact command it happens to me as well, so seems like a no-brainer. Thanks!

Actually, the files are deleted afterwards. They are not deleted for you? Maybe they should be aggregated to the single .coverage file.

@JelleZijlstra JelleZijlstra merged commit 77021f0 into psf:main Jun 9, 2021
@Jackenmen Jackenmen deleted the add_coverage_to_gitignore branch June 9, 2021 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JelleZijlstra JelleZijlstra JelleZijlstra approved these changes

@felix-hilden felix-hilden felix-hilden approved these changes

Assignees
No one assigned
Labels
skip news Pull requests that don't need a changelog entry.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Jackenmen @JelleZijlstra @felix-hilden @ichard26
0