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

django-debreach does not work with Django 3 #17

Closed
javierpena opened this issue Oct 9, 2019 · 7 comments
Closed

django-debreach does not work with Django 3 #17

javierpena opened this issue Oct 9, 2019 · 7 comments

Comments

@javierpena
Copy link
Contributor

Hi,

The Fedora project is updating the Django package to Django 3, and it looks like there is an incompatibility between django-debrach and the new version [1]. I am not sure if the issue is isolated to the usage of django.utils.six, or there is more to it.

Could you check this out? Thanks!

[1] - https://bugzilla.redhat.com/show_bug.cgi?id=1759926

@lpomfrey
Copy link
Owner

Sure, I'll take a look over the next couple of days and get a release out.

Thanks.

@lpomfrey
Copy link
Owner

This should be fixed in 2.0.0 which drops support for django < 2.0 and python 2.7 completely. Let me know if you have any issues.

Thanks. :)

@javierpena
Copy link
Contributor Author

That was fast! Thanks a lot.

Would it be possible to upload a tarball to PyPi, too? We need that to build the package, the wheel does not work for us.

@lpomfrey
Copy link
Owner

Oops, I thought I had already, sorry. Should be there now.

@javierpena
Copy link
Contributor Author

javierpena commented Oct 10, 2019

Awesome :). Now I'm still finding some troubles while running unit tests:

ERROR: debreach.tests (unittest.loader._FailedTest)

ImportError: Failed to import test module: debreach.tests
Traceback (most recent call last):
File "/usr/lib64/python3.7/unittest/loader.py", line 436, in _find_test_path
module = self._get_module_from_name(name)
File "/usr/lib64/python3.7/unittest/loader.py", line 377, in _get_module_from_name
import(name)
File "/tmp/django-debreach/debreach/tests.py", line 11, in
from debreach.decorators import append_random_comment, random_comment_exempt
File "/tmp/django-debreach/debreach/decorators.py", line 4, in
from django.utils.decorators import available_attrs, decorator_from_middleware
ImportError: cannot import name 'available_attrs' from 'django.utils.decorators' (/tmp/django-debreach/.eggs/Django-3.0a1-py3.7.egg/django/utils/decorators.py)

Do you think this is also related to Django 3?

@lpomfrey
Copy link
Owner

Ah, yea, that's definitely an oversight on my part. Try version 2.0.1 which should be on pypi now.

@javierpena
Copy link
Contributor Author

Yes, 2.0.1 works like a charm. Thanks again!

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

No branches or pull requests

2 participants