Repository navigation
fix: Loosen strict dependency pins for pyyaml and psutil - #295
Closed
shirasassoon wants to merge 2 commits into
Closed
shirasassoon wants to merge 2 commits into
shirasassoon wants to merge 2 commits into
Conversation
Loosen the exact pins pyyaml==6.0.2 and psutil==7.0.0 to compatible ranges (pyyaml>=6.0.2,<7 and psutil>=7,<8) so fabric-cli can co-resolve with newer fabric-cicd versions. fabric-cicd 1.4.0 requires pyyaml>=6.0.3, which conflicted with the exact pin and made installation unresolvable. Resolves microsoft#294 Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
The compatible ranges resolve the stated conflict while retaining major-version boundaries.
0 open findings
What changed in this PR
Loosens dependency constraints to support newer fabric-cicd releases.
Changes:
- Allows compatible PyYAML 6.x and psutil 7.x versions.
- Synchronizes development requirements.
- Adds a changie entry.
| File | Description |
|---|---|
pyproject.toml |
Updates runtime dependency ranges. |
requirements-dev.txt |
Mirrors dependency ranges. |
.changes/unreleased/fixed-20261013-164235.yaml |
Documents the fix. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
shirasassoon
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
fabric-cicdversions #294Summary
The exact pins
pyyaml==6.0.2andpsutil==7.0.0inpyproject.tomlconflict with newerfabric-cicdreleases.fabric-cicd1.4.0 requirespyyaml>=6.0.3, which cannot be satisfied alongside the exact pinpyyaml==6.0.2, making installation unresolvable when a newerfabric-cicdis pulled in (the project depends onfabric-cicd>=1.3.0with no upper bound).Changes
pyproject.toml:pyyaml==6.0.2->pyyaml>=6.0.2,<7andpsutil==7.0.0->psutil>=7,<8requirements-dev.txt: mirrored the same two ranges to keep it in sync with the manifestfixedchangie entry documenting the loosened pinsThe loosened ranges still exclude the next major of each package, and the APIs used (
yaml.safe_load/yaml.load/yaml.dumpandpsutil.Process()/.parent()/.pid) are stable across them.Validation
pip install -e .co-resolvesfabric-cicd 1.4.0+pyyaml 6.0.3+psutil 7.2.2(the previously-unresolvable scenario)tests/test_core,tests/test_utils,tests/test_parsers: 617 passed