-
-
Notifications
You must be signed in to change notification settings - Fork 98
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
Removes travis and updates supported Django versions #131
Conversation
…still work), as these versions no longer are supported by Django. I've done this to ensure this library is easily maintained before we start supporting ASGI.
What do you think about running codecov on both pull requests and pushes to master? The readme badge will display master coverage, but running it on pull-requests will post codecov reports in PRs. |
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, but see suggestion about codecov in PRs
Co-authored-by: Sondre Lillebø Gundersen <sondrelg@live.no>
Codecov Report
@@ Coverage Diff @@
## master #131 +/- ##
==========================================
+ Coverage 82.69% 82.90% +0.21%
==========================================
Files 7 8 +1
Lines 416 427 +11
==========================================
+ Hits 344 354 +10
- Misses 72 73 +1
Continue to review full report at Codecov.
|
Removes travis and updates supported Django versions
Thanks @sondrelg for starting the branch! ❤️
Removes Travis and implements GitHub actions.
Removes unsupported Django versions. Older versions may work (and they do), but are no longer officially supported. This is to make the library easily maintainable.
Prepares implementation of
pre-commit
,black
andisort
in GitHub actions, but this will come later.Closes #127