Skip to content

feat: Add constraints output type - #197

Merged
rapids-bot[bot] merged 3 commits into
rapidsai:mainfrom
KyleFromNVIDIA:output-type-constraints
Aug 5, 2026
Merged

feat: Add constraints output type#197
rapids-bot[bot] merged 3 commits into
rapidsai:mainfrom
KyleFromNVIDIA:output-type-constraints

Conversation

@KyleFromNVIDIA

Copy link
Copy Markdown
Member

Because constraints don't allow extras, sometimes we want to separate the generation of requirements and constraints. Add a constraints output type that is almost identical to requirements.

Fixes #196

Because constraints don't allow extras, sometimes we want to
separate the generation of requirements and constraints. Add a
`constraints` output type that is almost identical to `requirements`.

Fixes rapidsai#196
@KyleFromNVIDIA
KyleFromNVIDIA requested a review from a team as a code owner August 3, 2026 18:19
@KyleFromNVIDIA
KyleFromNVIDIA requested a review from gforsyth August 3, 2026 18:19
@KyleFromNVIDIA KyleFromNVIDIA added feature request New feature or request non-breaking Introduces a non-breaking change labels Aug 3, 2026
@bdice

bdice commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

This seems alright to me. It would make dependencies.yaml more expressive.

@jameslamb jameslamb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Small comments, no major concerns.

Agree a minor version bump (via feat:) should be enough.

rapids-dependency-file-generator should be able to process all existing dependencies.yaml files today, but the new version will be required to handle constraints outputs. Any places that are == pinned to an older rapids-dependency-file-generator will need updates, but I'm not aware of any of those and we should get a big loud error alerting us to that.

Comment thread src/rapids_dependency_file_generator/_config.py
Comment thread tests/test_rapids_dependency_file_generator.py Outdated
@KyleFromNVIDIA

Copy link
Copy Markdown
Member Author

/merge

@rapids-bot
rapids-bot Bot merged commit 0ce26bf into rapidsai:main Aug 5, 2026
5 checks passed
GPUtester pushed a commit that referenced this pull request Aug 5, 2026
# [1.22.0](v1.21.0...v1.22.0) (2026-08-05)

### Features

* Add `constraints` output type ([#197](#197)) ([0ce26bf](0ce26bf))
@GPUtester

Copy link
Copy Markdown

🎉 This PR is included in version 1.22.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

rapids-bot Bot pushed a commit to rapidsai/docker that referenced this pull request Aug 27, 2026
Fixes #911

NVIDIA/cudf#23556 and PRs like it getting merged means that this project needs to use a newer `rapids-dependency-file-generator`.

This proposes:

* bumping the floor to `1.22` (the first version with the necessary changes: rapidsai/dependency-file-generator#197)
* allowing `rapids-dependency-file-generator` to float within `1.*`

Authors:
  - James Lamb (https://github.com/jameslamb)

Approvers:
  - Gil Forsyth (https://github.com/gforsyth)

URL: #912
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature request New feature or request non-breaking Introduces a non-breaking change released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

New output type: constraints

4 participants