Merged
Conversation
Contributor
Contributor
🤖 Keepalive Loop StatusPR #133 | Agent: Needs-attention | Iteration 0/5 Current State
🔍 Failure Classification| Error type | infrastructure | |
Contributor
|
| Field | Value |
|---|---|
| Exit Code | 1 |
| Error Category | unknown |
| Error Type | codex |
| Run | View logs |
🔧 Suggested Recovery
Capture logs and context; retry once and escalate if the issue persists.
📝 What to do
- Check the workflow logs for detailed error output
- If this is a configuration issue, update the relevant settings
- If the error persists, consider adding the
needs-humanlabel for manual review - Re-run the workflow once the issue is resolved
Output summary
No output captured
Contributor
|
Autofix updated these files:
|
Contributor
There was a problem hiding this comment.
Pull request overview
Adjusts CI so PR Gate stays fast by skipping slow integration and release/packaging tests, while still running the full surface area on main and providing an on-demand/nightly workflow for release E2E validation.
Changes:
- Update PR Gate to run pytest with xdist, skip integration directories, and skip
release-marked tests; update main CI to run pytest with xdist args. - Introduce a new
release-e2e.ymlworkflow (nightly/manual/PR-label) that runs onlyrelease-marked tests. - Add
releasemarker usage in tests/config, addpytest-xdistto dev deps, and hardensitecustomize.pyto strip xdist CLI flags when xdist isn’t available.
Reviewed changes
Copilot reviewed 12 out of 14 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
tests/test_sitecustomize.py |
Adds tests covering xdist-arg stripping/keeping behavior. |
tests/test_release_spec.py |
Marks PyInstaller build test as release. |
tests/integration/test_packaged_executable_assets.py |
Marks integration packaged-exe asset tests as release. |
sitecustomize.py |
Detects xdist availability and conditionally strips -n/--dist flags. |
requirements.lock |
Updates lock content to include new dev dependency graph (incl. xdist). |
requirements-dev.lock |
Mirrors lock updates for dev dependency set. |
pyproject.toml |
Adds pytest-xdist and declares release marker. |
CLAUDE.md |
Documents CI policy (Gate vs main CI vs release E2E). |
AGENTS.md |
Documents CI policy and agent guidance for running release tests. |
.github/workflows/release-e2e.yml |
Adds workflow to run release tests nightly or via PR label. |
.github/workflows/pr-00-gate.yml |
Tunes Gate to skip integration + release tests and disable coverage/soft-gate. |
.github/workflows/ci.yml |
Adds xdist args to main CI pytest invocation. |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.
No description provided.