Fix linux requirements when pip installing#68779
Open
dwoz wants to merge 12 commits intosaltstack:3006.xfrom
Open
Fix linux requirements when pip installing#68779dwoz wants to merge 12 commits intosaltstack:3006.xfrom
dwoz wants to merge 12 commits intosaltstack:3006.xfrom
Conversation
dwoz
force-pushed
the
fix-pip-install-metadata
branch
from
dcd3bcf to
285cf83
Compare
This change moves core metadata to the [project] table in pyproject.toml, cleans up requirement files for PEP 517 compatibility, ensures dependencies are dynamically discovered from .txt files, updates static requirement files via pre-commit hooks, and inhibits automatic code rewriting hooks to maintain scope.
The documentation build needs all Salt dependencies to correctly import modules for autodoc. Adding requirements/crypto.txt ensures that cryptographic dependencies (like pycryptodomex) are available.
dwoz
force-pushed
the
fix-pip-install-metadata
branch
from
285cf83 to
5026551
Compare
Update *-crypto.txt files across all platforms and Python versions to ensure version consistency and resolve pre-commit hook discrepancies.
Synchronize with CI environment by applying formatting changes made by the black pre-commit hook.
Include base.txt and zeromq.txt in the Windows packaging requirement compilation hooks. This ensures that the static windows.txt requirement files contain all necessary dependencies for onedir builds, resolving failures in install_salt.ps1.
The myst-parser requirement for documentation builds on Python 3.9 requires mdit-py-plugins which in turn requires markdown-it-py < 3.0.0. Other packages like rich were pulling in markdown-it-py >= 3.0.0 on some platforms, causing resolution failures. This commit adds a Python version-specific constraint and regenerates the affected static requirement files.
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
1 participant
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.
What does this PR do?
What issues does this PR fix or reference?
Fixes
Previous Behavior
Remove this section if not relevant
New Behavior
Remove this section if not relevant
Merge requirements satisfied?
[NOTICE] Bug fixes or features added to Salt require tests.
Commits signed with GPG?
Yes/No