Make _get_extent a utils function

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Author: timtreis

.cruft.json

@@ -1,23 +1,23 @@
 {
-  "template": "https://github.com/scverse/cookiecutter-scverse",
-  "commit": "c21b82bf134f3a0f13db7482d4fb04ca1f562d59",
-  "checkout": "main",
-  "context": {
-    "cookiecutter": {
-      "project_name": "spatialdata-plot",
-      "package_name": "spatialdata_plot",
-      "project_description": "A very interesting piece of code",
-      "author_full_name": "scverse",
-      "author_email": "scverse",
-      "github_user": "scverse",
-      "project_repo": "https://github.com/scverse/spatialdata-plot",
-      "license": "BSD 3-Clause License",
-      "_copy_without_render": [
-        ".github/workflows/**.yaml",
-        "docs/_templates/autosummary/**.rst"
-      ],
-      "_template": "https://github.com/scverse/cookiecutter-scverse"
-    }
-  },
-  "directory": null
+    "template": "https://github.com/scverse/cookiecutter-scverse",
+    "commit": "c21b82bf134f3a0f13db7482d4fb04ca1f562d59",
+    "checkout": "main",
+    "context": {
+        "cookiecutter": {
+            "project_name": "spatialdata-plot",
+            "package_name": "spatialdata_plot",
+            "project_description": "A very interesting piece of code",
+            "author_full_name": "scverse",
+            "author_email": "scverse",
+            "github_user": "scverse",
+            "project_repo": "https://github.com/scverse/spatialdata-plot",
+            "license": "BSD 3-Clause License",
+            "_copy_without_render": [
+                ".github/workflows/**.yaml",
+                "docs/_templates/autosummary/**.rst"
+            ],
+            "_template": "https://github.com/scverse/cookiecutter-scverse"
+        }
+    },
+    "directory": null
 }

.github/ISSUE_TEMPLATE/bug_report.yml

@@ -0,0 +1,17 @@
+# Based on spatialdata
+codecov:
+    require_ci_to_pass: false
+
+coverage:
+    status:
+        project:
+            default:
+                # Require 1% coverage, i.e., always succeed
+                target: 1
+        patch: false
+        changes: false
+
+comment:
+    layout: "diff, flags, files"
+    behavior: once
+    require_base: false

.github/ISSUE_TEMPLATE/config.yml

@@ -25,4 +25,4 @@ no_warn_no_return = True
 
 show_error_codes = True
 show_column_numbers = True
-error_summary = True
+error_summary = True

.github/ISSUE_TEMPLATE/feature_request.yml

@@ -5,14 +5,9 @@ default_stages:
     - commit
     - push
 minimum_pre_commit_version: 2.16.0
+ci:
+    skip: []
 repos:
-    - repo: https://github.com/nbQA-dev/nbQA
-      rev: 1.7.0
-      hooks:
-          - id: nbqa-pyupgrade
-            args: [--py38-plus]
-          - id: nbqa-black
-          - id: nbqa-isort
     - repo: https://github.com/psf/black
       rev: 23.3.0
       hooks:
@@ -25,73 +20,14 @@ repos:
       rev: 1.13.0
       hooks:
           - id: blacken-docs
-    - repo: https://github.com/PyCQA/isort
-      rev: 5.12.0
-      hooks:
-          - id: isort
     - repo: https://github.com/pre-commit/mirrors-mypy
       rev: v1.2.0
       hooks:
           - id: mypy
             additional_dependencies: [numpy, types-requests]
-            exclude: tests/|docs/|temp/
-    - repo: https://github.com/asottile/yesqa
-      rev: v1.4.0
-      hooks:
-          - id: yesqa
-            additional_dependencies:
-                - flake8-tidy-imports
-                - flake8-docstrings
-                - flake8-rst-docstrings
-                - flake8-comprehensions
-                - flake8-bugbear
-                - flake8-blind-except
-    - repo: https://github.com/pre-commit/pre-commit-hooks
-      rev: v4.4.0
-      hooks:
-          - id: detect-private-key
-          - id: check-ast
-          - id: end-of-file-fixer
-          - id: mixed-line-ending
-            args: [--fix=lf]
-          - id: trailing-whitespace
-          - id: check-case-conflict
-          - id: check-docstring-first
-    - repo: https://github.com/PyCQA/autoflake
-      rev: v2.0.2
-      hooks:
-          - id: autoflake
-            args:
-                - --in-place
-                - --remove-all-unused-imports
-                - --remove-unused-variable
-                - --ignore-init-module-imports
-    - repo: https://github.com/PyCQA/flake8
-      rev: 6.0.0
-      hooks:
-          - id: flake8
-            additional_dependencies:
-                - flake8-tidy-imports
-                - flake8-docstrings
-                - flake8-rst-docstrings
-                - flake8-comprehensions
-                - flake8-bugbear
-                - flake8-blind-except
-    - repo: https://github.com/asottile/pyupgrade
-      rev: v3.3.1
-      hooks:
-          - id: pyupgrade
-            args: [--py3-plus, --py39-plus, --keep-runtime-typing]
-    - repo: https://github.com/pre-commit/pygrep-hooks
-      rev: v1.10.0
-      hooks:
-          - id: python-no-eval
-          - id: python-use-type-annotations
-          - id: python-check-blanket-noqa
-          - id: rst-backticks
-          - id: rst-directive-colons
-          - id: rst-inline-touching-normal
-    - repo: https://github.com/PyCQA/doc8
-      rev: v1.1.1
+            exclude: tests/|docs/
+    - repo: https://github.com/charliermarsh/ruff-pre-commit
+      rev: v0.0.261
       hooks:
-          - id: doc8
+          - id: ruff
+            args: [--fix, --exit-non-zero-on-fix]

.github/codecov.yml

@@ -6,11 +6,10 @@ build:
         python: "3.10"
 sphinx:
     configuration: docs/conf.py
-    # disable this for more lenient docs builds
-    fail_on_warning: true
+    fail_on_warning: false
 python:
     install:
         - method: pip
           path: .
           extra_requirements:
-              - doc
+              - docs