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 Coveralls integration #149

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

Add Coveralls integration #149

wants to merge 1 commit into from

Conversation

blag
Copy link
Collaborator

@blag blag commented May 21, 2024

Add Coveralls integration so we can track coverage over time, as well as enable contributors to see how their changes affect code coverage.

@PetrDlouhy
Copy link
Contributor

It is interesting, that Coverals report the Django <5.0 branch of _get_flatchoices untested:
Snímek obrazovky_2024-05-21_08-27-42

I am not sure why that happened.

@PetrDlouhy
Copy link
Contributor

@blag I think that there must be some kind of problem with the different Django versions.
While in Django 5.0 the tree looks like this:
Snímek obrazovky_2024-05-30_12-03-02

On Django 4.2 the tree looks like this:
Snímek obrazovky_2024-05-30_12-05-39
And the results are not accessible.

Maybe it uses different configuration with each Django version.

Also on Django 5.0 are examples part of the results, but I think they should not be.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants