ci: test the Python event source - #241
Open
ryanduguid wants to merge 4 commits into
Open
Conversation
ryanduguid
marked this pull request as ready for review
August 18, 2026 15:06
start-prism.sh ended with a non-backgrounded prism mock, so invoking it in the foreground never returned. Every push and PR sat on the Start Prism step until the 25 minute job timeout, which meant the readiness loop and pytest never ran. Background the last mock so the script returns once all nine are spawned and the readiness loop is what gates the Test step. pip-audit 2.10.1 declares Requires-Python >=3.10, so the 3.8 leg died while installing requirements/dev.txt. Gate it with an environment marker and run the audit once on the newest leg, since auditing requirements.txt does not vary by interpreter. Restore flake8-black so black checking comes back for every consumer of requirements/flake8.txt, including tox -e flake8, and drop the --select that was overriding the [flake8] section in setup.cfg. Pin black, because an unpinned black now reports 543 BLK100 findings against generated code on master and would make the restored gate unusable. Reformat the two test files that were genuinely misformatted so the full gate passes. Widen the matrix to the declared support range; setup.py and tox move to the same range in the runtime policy branch.
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.
Summary
masterpip checkee65f8bddb16f37cde3e32741d465eeed6507a74; update that SHA deliberately when the test contract changesValidation
actionlint .github/workflows/build-test-lint.ymlpython -m pip checkpython -m pip_audit -r requirements.txt— no known vulnerabilitiespython -m flake8 --select E9,F63,F7,F82 xero_pythonpython -m builddatetime.timestamp()portability failure was isolated and will be proposed separatelyThe CI audit tool is pinned to
pip-audit2.10.1. Other dependencies, including the CI-onlybuildtool, retain the repository's existing open requirement policy; this workflow is therefore not a hermetic dependency build and does not introduce a lock policy. The authoritative result for the Linux Python 3.8/3.12/3.13 matrix is this pull request's exact-head Actions run. The pull request remains a draft until those hosted checks finish.