From d90598fb3952edef784648344e683f559456ec18 Mon Sep 17 00:00:00 2001 From: James McKinney <26463+jpmckinney@users.noreply.github.com> Date: Wed, 4 Sep 2024 22:09:09 -0400 Subject: [PATCH] chore: Avoid E501 in flake8 and RUF100 in ruff astral-sh/ruff#7940 --- pelican/util/exchange_rates_db.py | 16 +++++++++------- pyproject.toml | 2 +- tests/dataset/test_metadata_aggregator.py | 4 ++-- 3 files changed, 12 insertions(+), 10 deletions(-) diff --git a/pelican/util/exchange_rates_db.py b/pelican/util/exchange_rates_db.py index 4dada4b7..8c63f482 100644 --- a/pelican/util/exchange_rates_db.py +++ b/pelican/util/exchange_rates_db.py @@ -63,15 +63,17 @@ def update_from_fixer_io() -> None: date_str = target_date.strftime("%Y-%m-%d") logger.info("Fetching exchange rates for %s.", date_str) - # Datlab had already downloaded the exchange rates to EUR from another project. Changing the base would require - # re-downloading decades of exchange rates. It makes no difference to the application's logic, as all currency - # operations are performed in USD. + # Datlab had already downloaded the exchange rates to EUR from another project. Changing the base would + # require re-downloading decades of exchange rates. It makes no difference to the application's logic, + # as all currency operations are performed in USD. # - # The Basic plan is required to request rates for all base currencies. The Professional plan supports the Time-Series - # Endpoint, which can request rates for multiple dates at once. https://fixer.io/documentation + # The Basic plan is required to request rates for all base currencies. The Professional plan supports + # the Time-Series Endpoint, which can request rates for multiple dates at once. + # https://fixer.io/documentation # - # "The Fixer API delivers EOD / End of Day historical exchange rates, which become available at 00:05am GMT for the - # previous day and are time stamped at one second before midnight." https://fixer.io/faq + # "The Fixer API delivers EOD / End of Day historical exchange rates, which become available at 00:05am + # GMT for the previous day and are time stamped at one second before midnight." + # https://fixer.io/faq response = requests.get( f"{BASE_URL}/{date_str}?access_key={access_key}&base=EUR&symbols={symbols}", timeout=10 ) diff --git a/pyproject.toml b/pyproject.toml index ea657265..b24df87b 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -11,7 +11,7 @@ select = ["ALL"] ignore = [ "ANN", "COM", "EM", # https://docs.astral.sh/ruff/formatter/#conflicting-lint-rules - "W191", "E501", "D206", "Q000", "Q001", "Q002", "Q003", "ISC001", + "W191", "D206", "Q000", "Q001", "Q002", "Q003", "ISC001", "D203", "D212", # ignore incompatible rules "D200", # documentation preferences "C901", "PLR091", # complexity preferences diff --git a/tests/dataset/test_metadata_aggregator.py b/tests/dataset/test_metadata_aggregator.py index adc1afbb..aa539428 100644 --- a/tests/dataset/test_metadata_aggregator.py +++ b/tests/dataset/test_metadata_aggregator.py @@ -192,7 +192,7 @@ def test_get_kingfisher_metadata_release( "name": "Open Contracting Partnership", }, "description": { - "en": "For providing overall process titles and descriptions, often to give a summary of the contracting process as a whole.", + "en": "For providing overall process titles and descriptions, often to give a summary of the contracting process as a whole.", # noqa: E501 }, "documentationUrl": {"en": "https://extensions.open-contracting.org/en/extensions/process_title/"}, "name": {"en": "Process level title and description"}, @@ -260,7 +260,7 @@ def test_get_kingfisher_metadata_record( "name": "Open Contracting Partnership", }, "description": { - "en": "For providing overall process titles and descriptions, often to give a summary of the contracting process as a whole.", + "en": "For providing overall process titles and descriptions, often to give a summary of the contracting process as a whole.", # noqa: E501 }, "documentationUrl": {"en": "https://extensions.open-contracting.org/en/extensions/process_title/"}, "name": {"en": "Process level title and description"},