🎨 black & isort (#45)

Author: techouse

.github/workflows/test.yml

@@ -2,12 +2,33 @@ name: Test
 
 on:
   push:
-    branches: [ master ]
+    branches:
+      - master
   pull_request:
-    branches: [ master ]
+    branches:
+      - master
+defaults:
+  run:
+    shell: bash
+permissions: read-all
 
 jobs:
+  analyze:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+      - name: Set up Python
+        uses: actions/setup-python@v3
+        with:
+          python-version: "3.x"
+      - name: Install dependencies
+        run: |
+          python3 -m pip install --upgrade pip
+          pip install -r requirements_dev.txt
+      - name: Run static analysis
+        run: tox -e linters
   test:
+    needs: analyze
     runs-on: ubuntu-latest
     strategy:
       matrix:

mysql_to_sqlite3/cli.py

@@ -19,9 +19,7 @@
     help="SQLite3 database file",
     required=True,
 )
-@click.option(
-    "-d", "--mysql-database", default=None, help="MySQL database name", required=True
-)
+@click.option("-d", "--mysql-database", default=None, help="MySQL database name", required=True)
 @click.option("-u", "--mysql-user", default=None, help="MySQL user", required=True)
 @click.option(
     "-p",
@@ -80,24 +78,16 @@
     help="Prefix indices with their corresponding tables. "
     "This ensures that their names remain unique across the SQLite database.",
 )
-@click.option(
-    "-X", "--without-foreign-keys", is_flag=True, help="Do not transfer foreign keys."
-)
+@click.option("-X", "--without-foreign-keys", is_flag=True, help="Do not transfer foreign keys.")
 @click.option(
     "-W",
     "--without-data",
     is_flag=True,
     help="Do not transfer table data, DDL only.",
 )
-@click.option(
-    "-h", "--mysql-host", default="localhost", help="MySQL host. Defaults to localhost."
-)
-@click.option(
-    "-P", "--mysql-port", type=int, default=3306, help="MySQL port. Defaults to 3306."
-)
-@click.option(
-    "-S", "--skip-ssl", is_flag=True, help="Disable MySQL connection encryption."
-)
+@click.option("-h", "--mysql-host", default="localhost", help="MySQL host. Defaults to localhost.")
+@click.option("-P", "--mysql-port", type=int, default=3306, help="MySQL port. Defaults to 3306.")
+@click.option("-S", "--skip-ssl", is_flag=True, help="Disable MySQL connection encryption.")
 @click.option(
     "-c",
     "--chunk",
@@ -123,9 +113,7 @@
 )
 @click.option("-q", "--quiet", is_flag=True, help="Quiet. Display only errors.")
 @click.option("--debug", is_flag=True, help="Debug mode. Will throw exceptions.")
-@click.version_option(
-    message=tabulate(info(), headers=["software", "version"], tablefmt="github")
-)
+@click.version_option(message=tabulate(info(), headers=["software", "version"], tablefmt="github"))
 def cli(
     sqlite_file,
     mysql_user,
@@ -153,9 +141,7 @@ def cli(
     """Transfer MySQL to SQLite using the provided CLI options."""
     try:
         if mysql_tables and exclude_mysql_tables:
-            raise click.UsageError(
-                "Illegal usage: --mysql-tables and --exclude-mysql-tables are mutually exclusive!"
-            )
+            raise click.UsageError("Illegal usage: --mysql-tables and --exclude-mysql-tables are mutually exclusive!")
 
         converter = MySQLtoSQLite(
             sqlite_file=sqlite_file,
@@ -167,8 +153,7 @@ def cli(
             limit_rows=limit_rows,
             collation=collation,
             prefix_indices=prefix_indices,
-            without_foreign_keys=without_foreign_keys
-            or (mysql_tables is not None and len(mysql_tables) > 0),
+            without_foreign_keys=without_foreign_keys or (mysql_tables is not None and len(mysql_tables) > 0),
             without_data=without_data,
             mysql_host=mysql_host,
             mysql_port=mysql_port,

mysql_to_sqlite3/click_utils.py

@@ -40,9 +40,7 @@ def parser_process(value, state):
             # call the actual process
             self._previous_parser_process(value, state)
 
-        retval = super(OptionEatAll, self).add_to_parser(  # pylint: disable=E1111
-            parser, ctx
-        )
+        retval = super(OptionEatAll, self).add_to_parser(parser, ctx)  # pylint: disable=E1111
         for name in self.opts:
             # pylint: disable=W0212
             our_parser = parser._long_opt.get(name) or parser._short_opt.get(name)

mysql_to_sqlite3/debug_info.py

@@ -44,9 +44,7 @@ def _implementation():
             sys.pypy_version_info.minor,  # noqa: ignore=E1101 pylint: disable=E1101
             sys.pypy_version_info.micro,  # noqa: ignore=E1101 pylint: disable=E1101
         )
-        rel = (
-            sys.pypy_version_info.releaselevel  # noqa: ignore=E1101 pylint: disable=E1101
-        )
+        rel = sys.pypy_version_info.releaselevel  # noqa: ignore=E1101 pylint: disable=E1101
         if rel != "final":
             implementation_version = "".join([implementation_version, rel])
     elif implementation == "Jython":

mysql_to_sqlite3/sqlite_utils.py

@@ -10,6 +10,7 @@
 import six
 from pytimeparse.timeparse import timeparse
 
+
 if version_info.major == 3 and 4 <= version_info.minor <= 6:
     from backports.datetime_fromisoformat import MonkeyPatch  # pylint: disable=E0401
 
@@ -60,12 +61,8 @@ def convert_date(value):
         try:
             return date.fromisoformat(value.decode())
         except ValueError as err:
-            raise ValueError(  # pylint: disable=W0707
-                "DATE field contains {}".format(err)
-            )
+            raise ValueError("DATE field contains {}".format(err))  # pylint: disable=W0707
     try:
         return datetime.strptime(value.decode(), "%Y-%m-%d").date()
     except ValueError as err:
-        raise ValueError(  # pylint: disable=W0707
-            "DATE field contains Invalid isoformat string: {}".format(err)
-        )
+        raise ValueError("DATE field contains Invalid isoformat string: {}".format(err))  # pylint: disable=W0707