Skip to content

[MNT] move frozen 2024 dependencies test to uv dependency freeze mechanism from dedicated depset#1070

Open
fkiraly wants to merge 4 commits into
mainfrom
deps-test
Open

[MNT] move frozen 2024 dependencies test to uv dependency freeze mechanism from dedicated depset#1070
fkiraly wants to merge 4 commits into
mainfrom
deps-test

Conversation

@fkiraly

@fkiraly fkiraly commented Jun 13, 2026

Copy link
Copy Markdown
Collaborator

This PR replaces the dedicated pyproject depset mechanism to test skpro against frozen 2024 dependencies by the uv native feature for frozen dependency sets.

Copilot AI review requested due to automatic review settings June 13, 2026 13:18
@fkiraly fkiraly added the maintenance Continuous integration, unit testing & package distribution label Jun 13, 2026

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

Note

Copilot was unable to run its full agentic suite in this review.

Updates CI to test against a “frozen 2024” dependency resolution using uv’s exclude-newer mechanism instead of a dedicated pinned depset in pyproject.toml.

Changes:

  • Removed dependencies_2024 extras group from pyproject.toml.
  • Updated GitHub Actions test workflow to install via uv pip with UV_EXCLUDE_NEWER.
  • Added _Pipeline tags for object_type / estimator_type.

Reviewed changes

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

File Description
skpro/regression/compose/_pipeline.py Adds estimator metadata tags to classify the pipeline as a proba regressor.
pyproject.toml Removes the pinned 2024 depset extras group in favor of uv freezing.
.github/workflows/test.yml Switches CI install from pip extras to uv pip with an exclude-newer date.

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

Comment on lines +303 to +309
run: >
uv pip install .[dev]
--exclude-newer-package "setuptools=false"
--exclude-newer-package "scikit-base=false"
env:
UV_SYSTEM_PYTHON: 1
UV_EXCLUDE_NEWER: "2024-09-01"
Comment thread skpro/regression/compose/_pipeline.py
Comment thread .github/workflows/test.yml Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Continuous integration, unit testing & package distribution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants