-
Notifications
You must be signed in to change notification settings - Fork 105
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
Reawaken development #204
Reawaken development #204
Conversation
874e50b
to
6c6ab4d
Compare
* Update tox matrix * Update Circle CI config * Update setup.py with newer Python versions
6c6ab4d
to
3eadb4c
Compare
3eadb4c
to
111c78f
Compare
235af88
to
ef195a1
Compare
@tim-schilling I'd welcome your review on this, too, if you don't mind |
9c1e81d
to
b00c829
Compare
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.
It looks good to me. I didn't run anything, just gave the code a once over. The issue I spotted wouldn't harm anything even.
Co-authored-by: Tim Schilling <[email protected]>
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.
r+wc - just noticed some string clean-up that isn't necessary but a nice-to-have to avoid awkward concatenations, presumably from the automated formatting.
2a99dc9
to
a5bf0b3
Compare
This changeset aims to jump start development on the project.
six
(resolves Remove package six #199)pep8
andflake8
forruff
pre-commit
Once this is merged to
main
, we can look at any outstanding PRs and merge those that are safe/OK to merge. We'll then look to do a3.8alpha
or3.8rc
release, before a formal3.8
release.The longer goal remains to get the project into a state where it can hopefully be absorbed into Django Core - see #135