docs: add Where quick-gate-python is used section - #48
Merged
Merged
Conversation
cleder/awesome-python-testing#108 lists quick-gate-python (PyPI pygate-ci) among Python testing tools (merged 2026-08-07, per ledger/contribution-derivation.json). Confirmed live: the merge ref matches the merged content on the target repo's default branch README, grepped directly. A listing, not an endorsement. No code changed. Test suite could not be collected in this environment (ModuleNotFoundError-style import errors — the package is not pip-installed here; pip install is outside this session's permitted command set), so this is a docs-only change verified by direct inspection and a live git-ref check rather than pytest. Performed by a Hermes Labs autonomous system for Rolando Bosch (roli-lpci).
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughREADME.md adds a ChangesREADME Attribution
Estimated code review effort: 1 (Trivial) | ~2 minutes Change: Other ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
The section heading read "Where quick-gate-python is used", but its single entry is a listing, not a use: cleder/awesome-python-testing#108 merged a link to quick-gate-python, and the bullet itself already says "lists quick-gate-python ... a listing, not an endorsement". The heading therefore contradicted the sentence beneath it and asserted adoption on a public surface, which the Hermes Labs convention forbids: a listing is never a use, and a merge is not adoption. "Listed in" is what the evidence supports and matches the heading already used in hermes-labs-ai/hermes-rubric#35, and now in lintlang#93 and little-canary#77, from the same wave. The bullet is unchanged — it was accurate. Repository check re-run on this tree (src layout, package not installed into the interpreter, so PYTHONPATH points at src): PYTHONPATH=src python3 -m pytest -q -> 1 failed, 184 passed, 5 deselected in 23.42s The single failure is tests/test_cli.py::TestCLIParsing::test_runtime_version_matches_distribution_metadata, which raises importlib.metadata.PackageNotFoundError because the distribution is not pip-installed in this environment. It is an artifact of running from a source tree, unrelated to this README-only change, and CI (which installs the package) is the authority on it. This push invalidates any CI run or review recorded against the earlier head e48ec04; both must be repeated. Performed by a Hermes Labs autonomous system for Rolando Bosch (roli-lpci).
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
2 participants
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
Adds a "Where quick-gate-python is used" section linking cleder/awesome-python-testing#108, which lists quick-gate-python (PyPI
pygate-ci) among Python testing tools, merged 2026-08-07 perledger/contribution-derivation.json.Why
The listing repo already links to quick-gate-python (confirmed by grepping the merged content on its default-branch README directly). quick-gate-python's own README did not link back. A listing, not an endorsement.
Check
Docs-only change (README.md). Test suite could not be collected in this environment (import errors — the package isn't pip-installed here, and
pip installis outside this session's permitted command set), so this is verified by direct inspection and the live git-ref check above rather than pytest.Risk
None to runtime code. Draft per standing process; merge follows the Hermes gate + reviewer PASS.
Hermes Labs autonomous system
Summary by CodeRabbit