-
-
Notifications
You must be signed in to change notification settings - Fork 142
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
[feature] Added support for Ubuntu 20.04 (#189) #207
Conversation
b2b664c
to
e8f6025
Compare
- switched to use /usr/bin/python3 by default - python-virtualenv > python3-virtualenv - build in travis using ubuntu 20.04 - add ubuntu 20.04 (focal) to meta info - handle redis 4/5 and dependencies - removed step to install python-wheel (installed by pip later) - simplified python3-virtualenv installation - pin channels_redis to 2.4 for redis 4 compatibility Closes #189
e8f6025
to
30c1c34
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.
Looks good, only one minor doubt below.
files/generate_django_secret_key.py
Outdated
@@ -1,4 +1,4 @@ | |||
#!/usr/bin/env python | |||
#!/usr/bin/env python3 |
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.
are you sure this is needed? The virtualenv should know the python version without the need of introducing this change
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.
Not sure now, but I probably made this change and then forgot to remove it 😁
@PabloCastellano the type should be feature instead of feat |
More info about that here: openwisp/openwisp-utils#110 (comment) |
02755e3
to
26654f0
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.
Fixed the minor issue 26654f0.
Thanks!
(this PR integrates the work done in #190)
Closes #189