From 0c90b740cc4b81fa73b8a9e6b7b45754045e0e94 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Herv=C3=A9=20Beraud?= Date: Tue, 2 Jun 2020 21:07:04 +0200 Subject: [PATCH] Stop to use the __future__ module. The __future__ module [1] was used in this context to ensure compatibility between python 2 and python 3. We previously dropped the support of python 2.7 [2] and now we only support python 3 so we don't need to continue to use this module and the imports listed below. Imports commonly used and their related PEPs: - `division` is related to PEP 238 [3] - `print_function` is related to PEP 3105 [4] - `unicode_literals` is related to PEP 3112 [5] - `with_statement` is related to PEP 343 [6] - `absolute_import` is related to PEP 328 [7] [1] https://docs.python.org/3/library/__future__.html [2] https://governance.openstack.org/tc/goals/selected/ussuri/drop-py27.html [3] https://www.python.org/dev/peps/pep-0238 [4] https://www.python.org/dev/peps/pep-3105 [5] https://www.python.org/dev/peps/pep-3112 [6] https://www.python.org/dev/peps/pep-0343 [7] https://www.python.org/dev/peps/pep-0328 Change-Id: I739502c3379364aafd2363c09d9a8d539100bf44 --- vitrage/persistency/service.py | 2 -- vitrage/storage/history_facade.py | 2 -- vitrage/storage/impl_sqlalchemy.py | 3 --- vitrage/storage/sqlalchemy/migration/alembic_migrations/env.py | 2 -- vitrage/utils/datetime.py | 2 -- 5 files changed, 11 deletions(-) diff --git a/vitrage/persistency/service.py b/vitrage/persistency/service.py index e2acd8cbd..61139b6ad 100644 --- a/vitrage/persistency/service.py +++ b/vitrage/persistency/service.py @@ -12,8 +12,6 @@ # License for the specific language governing permissions and limitations # under the License. -from __future__ import print_function - from datetime import timedelta from concurrent.futures import ThreadPoolExecutor diff --git a/vitrage/storage/history_facade.py b/vitrage/storage/history_facade.py index 6b270b7ab..07c606f7c 100644 --- a/vitrage/storage/history_facade.py +++ b/vitrage/storage/history_facade.py @@ -12,8 +12,6 @@ # License for the specific language governing permissions and limitations # under the License. -from __future__ import absolute_import - import pytz import sqlalchemy from sqlalchemy import and_ diff --git a/vitrage/storage/impl_sqlalchemy.py b/vitrage/storage/impl_sqlalchemy.py index 1462a5b96..ed049288c 100644 --- a/vitrage/storage/impl_sqlalchemy.py +++ b/vitrage/storage/impl_sqlalchemy.py @@ -12,9 +12,6 @@ # License for the specific language governing permissions and limitations # under the License. - -from __future__ import absolute_import - from oslo_config import cfg from oslo_db.sqlalchemy import session as db_session from oslo_log import log diff --git a/vitrage/storage/sqlalchemy/migration/alembic_migrations/env.py b/vitrage/storage/sqlalchemy/migration/alembic_migrations/env.py index 47e2aba00..a4fd1d206 100644 --- a/vitrage/storage/sqlalchemy/migration/alembic_migrations/env.py +++ b/vitrage/storage/sqlalchemy/migration/alembic_migrations/env.py @@ -10,8 +10,6 @@ # implied. # See the License for the specific language governing permissions and # limitations under the License. - -from __future__ import with_statement from alembic import context from logging.config import fileConfig from oslo_db.sqlalchemy import enginefacade diff --git a/vitrage/utils/datetime.py b/vitrage/utils/datetime.py index a48d0cb4d..9fd4d4963 100644 --- a/vitrage/utils/datetime.py +++ b/vitrage/utils/datetime.py @@ -12,8 +12,6 @@ # License for the specific language governing permissions and limitations # under the License. -from __future__ import absolute_import - from datetime import datetime from datetime import timedelta from dateutil import parser