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

Ignore report files created by pytest-html #2962

Merged
merged 1 commit into from
May 28, 2024
Merged

Conversation

happz
Copy link
Collaborator

@happz happz commented May 27, 2024

It's a plugin that's not installed by default, but it proved to be very useful when running large sets of tests, e.g. package manager unit tests. The created report spoils the git index.

Pull Request Checklist

  • implement the feature

@happz happz added the code | trivial A simple patch - a couple of lines, an easy-to-understand change, a typo fix. label May 27, 2024
@happz happz added this to the 1.34 milestone May 27, 2024
@lukaszachy lukaszachy added the status | ready for merge The only missing piece is to do the rebase the current 'main' and let the CI finish. label May 28, 2024
@lukaszachy
Copy link
Collaborator

NOTE: lets merge it manually once we get to merging this (no wasting time with CI)

It's a plugin that's not installed by default, but it proved to be very
useful when running large sets of tests, e.g. package manager unit
tests. The created report spoils the git index.
@happz
Copy link
Collaborator Author

happz commented May 28, 2024

Merging before functional checks complete, just a .gitignore change.

@happz happz merged commit cb3ded9 into main May 28, 2024
5 of 19 checks passed
@happz happz deleted the gitignore-pytest-html branch May 28, 2024 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code | trivial A simple patch - a couple of lines, an easy-to-understand change, a typo fix. status | ready for merge The only missing piece is to do the rebase the current 'main' and let the CI finish.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants