Setup CI and linting (#17)

* added .pre-commit-config, added ruff config to pyproj, removed requirements.txt

* adds daily test CI

* 3.09 -> 3.9 nit

Author: norlandrhagen

.github/workflows/main.yml

@@ -0,0 +1,50 @@
+name: CI
+
+on:
+  push:
+    branches: [ "main" ]
+    paths-ignore:
+    - 'docs/**'
+  pull_request:
+    branches: [ "main" ]
+    paths-ignore:
+    - 'docs/**'
+  schedule: # every day at noon
+    - cron: "0 12 * * *"
+env:
+  UV_HTTP_TIMEOUT: 500
+
+jobs:
+
+  test:
+    name: ${{ matrix.python-version }}-build
+    runs-on: ubuntu-latest
+    defaults:
+      run:
+        shell: bash -l {0}
+    strategy:
+      matrix:
+        python-version: ["3.9", "3.10", "3.11"]
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Setup Python
+        id: setup-python
+        uses: actions/setup-python@v5
+        with:
+          python-version: ${{ matrix.python-version }}
+
+      - name: Install uv
+        run: |
+          curl -LsSf https://astral.sh/uv/install.sh | sh
+          uv venv .venv
+          echo "VIRTUAL_ENV=.venv" >> $GITHUB_ENV
+
+      - name: Install cubed-xarray
+        run: |
+           uv pip install --system '.'
+           uv pip install --system '.[test]'
+
+      - name: Running Tests
+        run: |
+          python -m pytest .

.pre-commit-config.yaml

@@ -0,0 +1,26 @@
+# https://pre-commit.com/
+ci:
+  autoupdate_schedule: monthly
+repos:
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v4.6.0
+    hooks:
+      - id: trailing-whitespace
+      - id: end-of-file-fixer
+      - id: check-yaml
+
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    # Ruff version.
+    rev: "v0.5.6"
+    hooks:
+      # Run the linter.
+      - id: ruff
+        args: [ --fix ]
+      # Run the formatter.
+      - id: ruff-format
+
+  - repo: https://github.com/pre-commit/mirrors-mypy
+    rev: v1.10.0
+    hooks:
+      - id: mypy
+        exclude: "docs"

README.md

@@ -11,7 +11,7 @@ Interface for using [cubed](https://github.com/cubed-dev/cubed) with [xarray](ht
 
 ## Installation
 
-Install via pip 
+Install via pip
 
 `pip install cubed-xarray`
 
@@ -34,7 +34,7 @@ Xarray objects backed by cubed arrays can be created either by:
 In (2) and (3) the choice to use `cubed.Array` instead of `dask.array.Array` is made by passing the keyword argument `chunked_array_type='cubed'`.
 To pass arguments to the constructor of `cubed.Array` you should pass them via the dictionary `from_array_kwargs`, e.g. `from_array_kwargs={'spec': cubed.Spec(allowed_mem='2GB')}`.
 
-If cubed and cubed-xarray are installed but dask is not, then specifying `chunked_array_type` is not necessary, 
+If cubed and cubed-xarray are installed but dask is not, then specifying `chunked_array_type` is not necessary,
 as the entrypoints system will then default to the only chunked parallel backend available (i.e. cubed).
 
 ## Sharp Edges 🔪

pyproject.toml

@@ -21,11 +21,22 @@ classifiers = [
 ]
 requires-python = ">=3.9"
 dependencies = [
-    "numpy >= 1.17",
+    "numpy >= 1.17.1",
     "xarray >= 2024.02.0",
     "cubed >= 0.14.2",
 ]
 
+
+
+[project.optional-dependencies]
+test = [
+    "pre-commit",
+    "ruff",
+    "pytest-mypy",
+    "pytest-cov",
+    "pytest"]
+
+
 [project.urls]
 homepage = "https://github.com/xarray-contrib/cubed-xarray"
 documentation = "https://github.com/xarray-contrib/cubed-xarray#readme"
@@ -50,9 +61,47 @@ fallback_version = "999"
 [tool.pytest.ini_options]
 junit_family = "xunit2"
 
-[tool.isort]
-profile = "black"
-skip_gitignore = "true"
-force_to_top = "true"
-default_section = "THIRDPARTY"
-known_first_party = "cubed_xarray"
+
+[mypy]
+files = "cubed_xarray/**/*.py"
+show_error_codes = true
+
+[tool.ruff]
+# Same as Black.
+line-length = 88
+indent-width = 4
+target-version = "py39"
+
+exclude = [
+    "docs",
+    ".eggs"]
+
+[tool.ruff.lint]
+# Enable Pyflakes (`F`) and a subset of the pycodestyle (`E`)  codes by default.
+# Unlike Flake8, Ruff doesn't enable pycodestyle warnings (`W`) or
+# McCabe complexity (`C901`) by default.
+select = ["E4", "E7", "E9", "F", "I"]
+per-file-ignores = {}
+
+# E402: module level import not at top of file
+# E731: do not assign a lambda expression, use a def
+# W503: line break before binary operator - not implimeted by ruff due to conflict with PEP8.
+
+ignore = ["E402", "E731"]
+
+# Allow fix for all enabled rules (when `--fix`) is provided.
+fixable = ["ALL"]
+unfixable = []
+
+[tool.ruff.format]
+# Like Black, use double quotes for strings.
+quote-style = "double"
+# Indent with spaces, rather than tabs.
+indent-style = "space"
+# Respect magic trailing commas.
+skip-magic-trailing-comma = false
+# Automatically detect the appropriate line ending.
+line-ending = "auto"
+
+[tool.ruff.lint.isort]
+known-first-party = ["cubed_xarray"]