-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Added note warning about using sqlite 3.26.0 in development #5491
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
Added a note in the developer guide which asks the developer to use python version 3.6.7 instead of python 3.6.8 which would use a higher version of sqlite3 giving rise to issur readthedocs#5477
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.
I would put on the note to be aware that there's an issue with the sqlite version
>= 3.26.0
but not refer to the python version probably
I agree with @raulcd on this.
My idea with adding a note here is to give the quick solution to the user if they hit this problem. How we can guide the user in this direction? I mean, what the user should do if they are in the situation described by @raulcd in the previous example?
docs/install.rst
Outdated
@@ -11,6 +11,12 @@ First, obtain `Python 3.6`_ and virtualenv_ if you do not already have them. | |||
Using a virtual environment is strongly recommended, | |||
since it will help you to avoid clutter in your system-wide libraries. | |||
|
|||
.. note:: |
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.
This should probably be a warning
instead.
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.
I'd really like to keep these edge cases out of the default flow for the install guide, otherwise it just gets messy. We should add them to a troubleshooting section at the bottom of the guide, with the error that they fix.
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.
The thing is that it's not an edge case.
All users installing Python 3.6 with an up to date system will hit this problem. The issue is fixed on Django 2.x but we are using 1.11.x. So, all new contributors will hit it until we upgrade our django version.
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.
OK, thanks sense then.
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.
OK, thanks sense then.
Added a note in the developer guide which asks the developer to use python version 3.6.7 instead of python 3.6.8 which would use a higher version of sqlite3 giving rise to issur #5477