search

modlinltd / django-advanced-filters

Add advanced filtering abilities to Django admin
https://pypi.org/project/django-advanced-filters/
MIT License
771 stars 173 forks source link

Update Dependencies and CI/CD Setup #109

Closed alonraiz closed 4 years ago

coveralls commented 5 years ago

Coverage Status

Coverage remained the same at 95.914% when pulling 5d6a99ddb2a9867b6b58d74889992fe2f5647ef9 on alonraiz:fix-and-update-ci into ba51e6946d1652796a82b2b95cceffbe1190a227 on modlinltd:develop.

coveralls commented 5 years ago

Coverage Status

Coverage remained the same at 95.914% when pulling 5d6a99ddb2a9867b6b58d74889992fe2f5647ef9 on alonraiz:fix-and-update-ci into ba51e6946d1652796a82b2b95cceffbe1190a227 on modlinltd:develop.

asfaltboy commented 4 years ago

Hey @alonraiz can you please add a bit more detail about the PR? What does it do and why?

alonraiz commented 4 years ago

Hey @asfaltboy, the original goal of this PR was to make the master branch pass again (since the current master will fail if you trigger it now). after seeing the Travis configuration file I took the liberty of simplifying it, adding more relevant environments and removing deprecated ones (like 3.3)

asfaltboy commented 4 years ago

Thanks a lot, @alonraiz, and apologies for the delay.

I took the liberty to fix some of the tests (most likely failed due to pytest/Django versions), and update the .travis.yml to use tox-travis, using your matrix as the basis.