From 18c271101d12eaa360c156cae0c5de5ac0949c50 Mon Sep 17 00:00:00 2001 From: David Wobrock Date: Sun, 20 Oct 2019 22:23:45 +0200 Subject: [PATCH] Refactor: remove redundant function call --- django_migration_linter/sql_analyser/base.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/django_migration_linter/sql_analyser/base.py b/django_migration_linter/sql_analyser/base.py index 0d01f2e..a7d8fb1 100644 --- a/django_migration_linter/sql_analyser/base.py +++ b/django_migration_linter/sql_analyser/base.py @@ -126,13 +126,12 @@ class BaseAnalyser(object): def _test_sql(self, test, sql): if test["fn"](sql, errors=self.errors): + err = build_error_dict(migration_test=test, sql_statement=sql) if test["code"] in self.exclude_migration_tests: logger.debug("Testing %s -- IGNORED", sql) - err = build_error_dict(migration_test=test, sql_statement=sql) self.ignored.append(err) else: logger.debug("Testing %s -- ERROR", sql) - err = build_error_dict(migration_test=test, sql_statement=sql) self.errors.append(err) else: logger.debug("Testing %s -- PASSED", sql)