From cb8492d8a777dfc32ef5d211989ffb250cad95dc Mon Sep 17 00:00:00 2001 From: Nikita Karamov Date: Sun, 13 Nov 2022 13:38:16 +0100 Subject: [PATCH] Use a context manager to write to /dev/null --- django_migration_linter/migration_linter.py | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/django_migration_linter/migration_linter.py b/django_migration_linter/migration_linter.py index c49680d..fabb997 100644 --- a/django_migration_linter/migration_linter.py +++ b/django_migration_linter/migration_linter.py @@ -302,15 +302,15 @@ class MigrationLinter: def get_sql(self, app_label, migration_name): logger.info(f"Calling sqlmigrate command {app_label} {migration_name}") - dev_null = open(os.devnull, "w") try: - sql_statement = call_command( - "sqlmigrate", - app_label, - migration_name, - database=self.database, - stdout=dev_null, - ) + with open(os.devnull, "w") as dev_null: + sql_statement = call_command( + "sqlmigrate", + app_label, + migration_name, + database=self.database, + stdout=dev_null, + ) except (ValueError, ProgrammingError): logger.warning( "Error while executing sqlmigrate on (%s, %s).",