Skip to content

Updating xarray package#86

Merged
spiani merged 1 commit into
mainfrom
updating_xarray
Apr 16, 2026
Merged

Updating xarray package#86
spiani merged 1 commit into
mainfrom
updating_xarray

Conversation

@spiani

@spiani spiani commented Apr 16, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings April 16, 2026 17:20
@spiani spiani self-assigned this Apr 16, 2026
@spiani spiani merged commit ca10263 into main Apr 16, 2026
6 checks passed
@spiani spiani deleted the updating_xarray branch April 16, 2026 17:22

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates dependency constraints and lockfile to support newer xarray and refreshed dev tooling versions.

Changes:

  • Relaxed xarray dependency constraint in pyproject.toml.
  • Updated dev dependencies (pre-commit, deptry, pytest) in pyproject.toml.
  • Regenerated poetry.lock, pulling in newer resolved versions (including xarray 2026.4.0) and new transitive deps (e.g., tomli).

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 2 comments.

File Description
pyproject.toml Updates dependency constraints for xarray and dev tooling.
poetry.lock Updates resolved versions to match new constraints (incl. new transitive deps).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread pyproject.toml
numpy = "^2.1.1"
matplotlib = "^3.10.0"
xarray = "^2025.1.1"
xarray = ">=2025.1.1"

Copilot AI Apr 16, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

xarray is now specified as a lower-bound-only constraint (">=") which means users installing from sdist/wheel can end up with arbitrarily newer xarray releases than the one validated in this repo (the lock already resolves to 2026.4.0). If you want the update but still want predictable installs, consider using a compatible range (e.g., caret) or adding an explicit upper bound.

Suggested change
xarray = ">=2025.1.1"
xarray = ">=2025.1.1,<2027.0.0"

Copilot uses AI. Check for mistakes.
Comment thread pyproject.toml
deptry = "^0.21.2"
pytest = "^8.3.4"
pre-commit = "^4.5"
deptry = ">0.21.2"

Copilot AI Apr 16, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

deptry = ">0.21.2" excludes exactly 0.21.2. If the intent is simply to relax the previous ^0.21.2 cap so newer deptry versions are allowed, prefer >=0.21.2 (or another bounded range) to avoid an unnecessary/accidental gap.

Suggested change
deptry = ">0.21.2"
deptry = ">=0.21.2"

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants