From f252c60f12ecbe915705570d886a4c13c58a904f Mon Sep 17 00:00:00 2001 From: Kamil Raczycki Date: Tue, 22 Oct 2024 09:52:12 +0200 Subject: [PATCH] fix: change default bool value --- .pre-commit-config.yaml | 12 ++++-------- overturemaestro/data_downloader.py | 1 + overturemaestro/release_index.py | 2 +- 3 files changed, 6 insertions(+), 9 deletions(-) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 77a0b63..64f2d19 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -12,19 +12,15 @@ repos: hooks: - id: ruff args: [--fix, --exit-non-zero-on-fix, --show-fixes] + # TODO: Switch back to upstream docformatter # after https://github.com/PyCQA/docformatter/issues/289 is fixed - - repo: https://github.com/s-weigand/docformatter - rev: 1ec30b7 + - repo: https://github.com/PyCQA/docformatter + rev: eb1df34 + # rev: v1.7.5 hooks: - id: docformatter additional_dependencies: [tomli] args: [--in-place, --config, ./pyproject.toml] - # - repo: https://github.com/PyCQA/docformatter - # rev: v1.7.5 - # hooks: - # - id: docformatter - # additional_dependencies: [tomli] - # args: ["--in-place", "--config", "./pyproject.toml"] - repo: https://github.com/pre-commit/mirrors-mypy rev: v1.11.1 hooks: diff --git a/overturemaestro/data_downloader.py b/overturemaestro/data_downloader.py index 0a2a334..65d0f94 100644 --- a/overturemaestro/data_downloader.py +++ b/overturemaestro/data_downloader.py @@ -42,6 +42,7 @@ def download_data_for_multiple_types( working_directory: Union[str, Path] = "files", ) -> list[Path]: ... + @overload def download_data_for_multiple_types( theme_type_pairs: list[tuple[str, str]], diff --git a/overturemaestro/release_index.py b/overturemaestro/release_index.py index 065b8de..c114f72 100644 --- a/overturemaestro/release_index.py +++ b/overturemaestro/release_index.py @@ -189,7 +189,7 @@ def load_release_index( *, geometry_filter: Optional[BaseGeometry] = None, remote_index: bool = False, - skip_index_download: bool = True, + skip_index_download: bool = False, ) -> gpd.GeoDataFrame: """ Load release index as a GeoDataFrame.