more linter failures and fix some from before

cal-itp · Oct 26, 2023 · b5bce36 · b5bce36
1 parent 0727a5b
commit b5bce36
Show file tree

Hide file tree

Showing 8 changed files with 10 additions and 10 deletions.
diff --git a/airflow/dags/download_gtfs_schedule_v2/email_download_failures.py b/airflow/dags/download_gtfs_schedule_v2/email_download_failures.py
@@ -30,7 +30,7 @@ def email_failures(task_instance: TaskInstance, execution_date, **kwargs):
     The following agency GTFS feeds could not be extracted on {execution_date}:
 
     {html_report}
-    """  # noqa: E231
+    """  # noqa: E231,E241
 
     if os.environ["AIRFLOW_ENV"] == "development":
         print(

diff --git a/airflow/plugins/operators/littlepay_raw_sync.py b/airflow/plugins/operators/littlepay_raw_sync.py
@@ -117,8 +117,8 @@ def sync_file(
         )
         try:
             metadata_str = fs.getxattr(
-                path=f"gs://{fileinfo.name}",
-                attr=PARTITIONED_ARTIFACT_METADATA_KEY,  # noqa: E231
+                path=f"gs://{fileinfo.name}",  # noqa: E231
+                attr=PARTITIONED_ARTIFACT_METADATA_KEY,
             )
         except KeyError:
             print(f"metadata missing on {fileinfo.name}")

diff --git a/jobs/gtfs-aggregator-scraper/gtfs_aggregator_scraper.py b/jobs/gtfs-aggregator-scraper/gtfs_aggregator_scraper.py
@@ -183,7 +183,7 @@ def main(aggregator: GTFSFeedAggregator, dry_run: bool = False, progress: bool =
     )
     if dry_run:
         typer.secho(
-            f"dry run; skipping upload of {humanize.naturalsize(len(scrape_result.content))}"
+            f"dry run; skipping upload of {humanize.naturalsize(len(scrape_result.content))}"  # noqa: E702
         )
     else:
         typer.secho(

diff --git a/jobs/gtfs-rt-parser-v2/gtfs_rt_parser.py b/jobs/gtfs-rt-parser-v2/gtfs_rt_parser.py
@@ -769,7 +769,7 @@ def main(
         typer.secho(f"missing: {files_missing_metadata}")
         typer.secho(f"invalid: {files_invalid_metadata}")
         raise RuntimeError(
-            f"too many files have missing/invalid metadata; {total - len(files)} of {total}"
+            f"too many files have missing/invalid metadata; {total - len(files)} of {total}"  # noqa: E702
         )
 
     if not files:
@@ -893,7 +893,7 @@ def main(
 
     if exceptions:
         exc_str = "\n".join(str(tup) for tup in exceptions)
-        msg = f"got {len(exceptions)} exceptions from processing {len(aggregations_to_process)} feeds:\n{exc_str}"
+        msg = f"got {len(exceptions)} exceptions from processing {len(aggregations_to_process)} feeds:\n{exc_str}"  # noqa: E231
         typer.secho(msg, err=True, fg=typer.colors.RED)
         raise RuntimeError(msg)
 

diff --git a/jobs/gtfs-schedule-validator/gtfs_schedule_validator_hourly.py b/jobs/gtfs-schedule-validator/gtfs_schedule_validator_hourly.py
@@ -435,7 +435,7 @@ def validate_hour(
         success_rate = len(result.successes) / len(extracts)
         if success_rate < GTFS_VALIDATE_LIST_ERROR_THRESHOLD:
             exc_str = "\n".join(str(tup) for tup in exceptions)
-            msg = f"got {len(exceptions)} exceptions from validating {len(extracts)} extracts:\n{exc_str}"
+            msg = f"got {len(exceptions)} exceptions from validating {len(extracts)} extracts:\n{exc_str}"  # noqa: E231
             if exceptions:
                 typer.secho(msg, err=True, fg=typer.colors.RED)
             raise RuntimeError(msg)

diff --git a/packages/calitp-data-analysis/calitp_data_analysis/geography_utils.py b/packages/calitp-data-analysis/calitp_data_analysis/geography_utils.py
@@ -168,7 +168,7 @@ def cut_segments(
         .drop(columns="temp_index")
     )
 
-    segmented2 = gpd.GeoDataFrame(segmented, crs=f"EPSG:{EPSG_CODE}")
+    segmented2 = gpd.GeoDataFrame(segmented, crs=f"EPSG:{EPSG_CODE}")  # noqa: E231
 
     return segmented2
 

diff --git a/packages/calitp-data-analysis/calitp_data_analysis/sql.py b/packages/calitp-data-analysis/calitp_data_analysis/sql.py
@@ -27,7 +27,7 @@ def get_engine(max_bytes=None, project="cal-itp-data-infra"):
     # Note that we should be able to add location as a uri parameter, but
     # it is not being picked up, so passing as a separate argument for now.
     return create_engine(
-        f"bigquery://{project}/?maximum_bytes_billed={max_bytes}",
+        f"bigquery://{project}/?maximum_bytes_billed={max_bytes}",  # noqa: E231
         location=CALITP_BQ_LOCATION,
         credentials_path=cred_path,
     )

diff --git a/packages/calitp-data-analysis/calitp_data_analysis/tables.py b/packages/calitp-data-analysis/calitp_data_analysis/tables.py
@@ -109,7 +109,7 @@ def _repr_html_(self):
                 </tr>
                 {table_body_html}
             </table>
-            """
+            """  # noqa: E221,E222
 
 
 tbls = AutoTable(