diff mbox

[2/3] Add sqlparse to production requirements.

Message ID 1470717525-22872-3-git-send-email-dja@axtens.net
State Changes Requested
Headers show

Commit Message

Daniel Axtens Aug. 9, 2016, 4:38 a.m. UTC
It's required for migration #10 (w/ RunSQL)

I have put a note in here to remind me to check what the deal is
with versions of sqlparse. I have stuck to <0.2 for now because
it's the conservative solution.

Signed-off-by: Daniel Axtens <dja@axtens.net>

---

Being production requirements, I haven't been able to test this
properly. I can do more comprehensive testing if anyone is worried.
---
 requirements-prod.txt | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Stephen Finucane Aug. 21, 2016, 2:16 p.m. UTC | #1
On 09 Aug 14:38, Daniel Axtens wrote:
> It's required for migration #10 (w/ RunSQL)
> 
> I have put a note in here to remind me to check what the deal is
> with versions of sqlparse. I have stuck to <0.2 for now because
> it's the conservative solution.
> 
> Signed-off-by: Daniel Axtens <dja@axtens.net>

Reviewed-by: Stephen Finucane <stephenfinucane@hotmail.com>
diff mbox

Patch

diff --git a/requirements-prod.txt b/requirements-prod.txt
index 2c88c7dca5ee..52f1af4de8d8 100644
--- a/requirements-prod.txt
+++ b/requirements-prod.txt
@@ -1,2 +1,7 @@ 
 Django>=1.8,<1.10
 psycopg2==2.6.1
+# todo: verify that 0.2 (released 2016-07-20) works
+#       cf the test requirements where:
+#       django-dev-toolbar 1.4 breaks with 0.2
+#                          1.5 fixes the breakage but breaks Django 1.7
+sqlparse>=0.1,<0.2