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

Python 3.7 #310

Merged
merged 25 commits into from
Aug 14, 2018
Merged

Python 3.7 #310

merged 25 commits into from
Aug 14, 2018

Conversation

Koed00
Copy link
Owner

@Koed00 Koed00 commented Aug 5, 2018

Removes support for Python 2 and older Django versions.
Renames async to async_task to prevent reserved keyword conflicts in Python 3.7

@codecov-io
Copy link

codecov-io commented Aug 5, 2018

Codecov Report

Merging #310 into master will increase coverage by 0.49%.
The diff coverage is 98.76%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #310      +/-   ##
==========================================
+ Coverage   89.88%   90.38%   +0.49%     
==========================================
  Files          44       43       -1     
  Lines        2866     2838      -28     
==========================================
- Hits         2576     2565      -11     
+ Misses        290      273      -17
Impacted Files Coverage Δ
django_q/conf.py 74.75% <ø> (+3.32%) ⬆️
django_q/tests/test_brokers.py 70.47% <ø> (-0.12%) ⬇️
django_q/tests/test_monitor.py 100% <100%> (ø) ⬆️
django_q/tests/test_cached.py 100% <100%> (ø) ⬆️
django_q/admin.py 100% <100%> (ø) ⬆️
django_q/__init__.py 100% <100%> (+50%) ⬆️
django_q/models.py 97.4% <100%> (+1.29%) ⬆️
django_q/tests/test_admin.py 100% <100%> (+5.55%) ⬆️
django_q/tasks.py 99.46% <100%> (ø) ⬆️
django_q/tests/test_cluster.py 100% <100%> (ø) ⬆️
... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2040771...239345a. Read the comment docs.

@Koed00 Koed00 merged commit 35ba23d into master Aug 14, 2018
@Koed00 Koed00 deleted the python37 branch August 16, 2018 10:13
KlaasH added a commit to azavea/pfb-network-connectivity that referenced this pull request Sep 9, 2019
"async" is a reserved word in Python 3.7, so django-q had to rename their
thing. Unfortunately they didn't make documentation or CHANGELOG changes
to go with the code changes and releases, so see
Koed00/django-q#306 and
Koed00/django-q#310 for more information.
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

Successfully merging this pull request may close these issues.

3 participants