From 7a7070d82a83602aa0745524f96d5d9f114d38a3 Mon Sep 17 00:00:00 2001 From: jonathan vanasco Date: Tue, 30 Jun 2020 12:31:58 -0400 Subject: [PATCH] added semicolon (;) to improve warning message clarity ### Description Added a semicolon to improve the clarity of warning message. I actually had a table named `backend`, and thought it was involved! While updating the code, I noticed no test that directly tests for this warning message. There are tests for the `Can't sort tables for DROP;` prefix of this message and the `exc.CircularDependencyError`; and some tests for the `exc.CircularDependencyError` message itself. I couldn't find any test for this particular message though. (Just thought I'd bring that up) No issue created, because this is minor. Closes: #5431 Pull-request: https://github.com/sqlalchemy/sqlalchemy/pull/5431 Pull-request-sha: 87fb5733ff4fc1a13dd94277716814ea852f654c Change-Id: I87a504d30a7dd5155c34f7d7f30b2116d0d3cd3f (cherry picked from commit b421c51b94fdef2bc542b0a40bfb21490b23ca15) --- lib/sqlalchemy/sql/ddl.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/sqlalchemy/sql/ddl.py b/lib/sqlalchemy/sql/ddl.py index d775322311..3b16394dfd 100644 --- a/lib/sqlalchemy/sql/ddl.py +++ b/lib/sqlalchemy/sql/ddl.py @@ -903,7 +903,7 @@ class SchemaDropper(DDLBase): util.warn( "Can't sort tables for DROP; an " "unresolvable foreign key " - "dependency exists between tables: %s, and backend does " + "dependency exists between tables: %s; and backend does " "not support ALTER. To restore at least a partial sort, " "apply use_alter=True to ForeignKey and " "ForeignKeyConstraint " -- 2.47.3