Release/0.5.0#31
Merged
Merged
Conversation
Adds a -r/--recipient CLI flag so users can encrypt secrets using only an age public key (age1...), without needing the private identity. Can be combined with -i; rejected with passphrase options or in decrypt mode.
Replaces rye with uv as the project's package and environment manager. - pyproject.toml: move [tool.rye] dev-dependencies to [dependency-groups].dev (PEP 735); bump requires-python from 3.8 to 3.10 to match classifiers and satisfy uv's stricter resolver. - Add salt's runtime dependencies explicitly to the dev group. The PyPI salt wheel does not declare them in its metadata, so uv (unlike rye) cannot pull them transitively. - Replace rye lock files with uv.lock. - CI workflows: switch to astral-sh/setup-uv and uv-native commands. - README: update development and release instructions. - Drop ANN101/ANN102 from ruff ignore list (removed in newer ruff), and apply ruff's auto-fixes across tests, typings, and renderers.
- Replace `Any` with `object` in `secure_value.py` and `renderers/age.py`, with a narrowing cast after `is_secure_value()`. - Replace `Any` with `object` in test fixture return types. - Replace `lambda _key: None` with a typed `_config_get_none` function in the two env-based renderer tests. - Rename `typings/pyrage/passphrase.py` and `x25519.py` to `.pyi` so stub-only files no longer trigger unused-parameter warnings; drop the now-redundant quoted forward refs. - Scope `reportExplicitAny = "none"` to `typings/` only — Salt and saltfactories APIs legitimately need `Any`.
Salt installs a DeferredStreamHandler on the root logger that buffers records and flushes them at interpreter shutdown. Under pytest the buffered records were flushed to streams capsys had already closed, producing spurious "I/O operation on closed file" tracebacks after the test summary. - configure_logging now attaches a StreamHandler to the saltstack_age package logger and sets propagate = False, so CLI records never reach the root logger. - tests/conftest.py drops salt's DeferredStreamHandler from the root logger at session start, eliminating the saltfactories-emitted record that was also caught by the same flush path.
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.
No description provided.