-
-
Notifications
You must be signed in to change notification settings - Fork 458
Replacing optional-dependencies with dependency-groups #2399
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
The contribution documentation needs to be changed to describe how to install the dependency groups |
This PR cannot be merged immediately due to a dependency group compatibility issue with the remote CI pipelines. Problem The changes in this PR migrate from optional-dependencies to modern PEP 735 dependency-groups in pyproject.toml. However, the current remote CI infrastructure does not support the newer dependency group syntax, which cause pipeline failures. Migration Strategy To safely implement this change, we need a 5-step approach: Step 0: ✅ (Current PR)
Step 1: Change to local CI pipelines where dependency-groups are used instead to install all dependencies Step 2: Merge This PR ⏳ Step 3: Change back to remote CI pipelines. Step 4: Change documentation to reflect the changes in the way dependencies are being installed. |
Another pre-requisite that occurred to me: We need |
Summary
Changes
Benefits
Test plan
Fixes #2387
PR Checklist: