-
Notifications
You must be signed in to change notification settings - Fork 299
Add Django 2.2 support #640
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
Conversation
Codecov Report
Merging #640 into master will not change coverage.
The diff coverage isn/a
.
@@ Coverage Diff @@ ## master #640 +/- ## ======================================= Coverage 96.16% 96.16% ======================================= Files 56 56 Lines 2866 2866 ======================================= Hits 2756 2756 Misses 110 110
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 c77fe0f...77176f3. Read the comment docs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sliverc woops I think I merged this too soon. Don't you need to add stuff to tox.ini envlist as well?
@n2ygk Indeed. I will follow up with another PR>
Description of the Change
Checklist
CHANGELOG.md
updated (only for user relevant changes)AUTHORS