Additional weekly CI fixes & improvements - #249
Merged
Merged
Conversation
Separate wheel installation from the headless GUI test step in the release workflow. The wheel is installed and a temporary test directory is created first, then pytest runs from that directory via the action's `working-directory` and `shell` settings so the shipped wheel tests run against the installed package instead of the source tree.
Remove the manual `force_deploy` workflow input and simplify the release conditions so the build and deploy jobs only run for `v*` tags. This prevents manual dispatches from attempting package publication from untagged commits, which would fail because PyPI rejects the local versions generated by `setuptools_scm`.
Expand the weekly checks failure issue workflow to include per-job results, clearer run links, and step-by-step recovery guidance. New issues are now assigned to the regular maintainers, while repeated failures are added as comments with the latest status summary.
Allow manual runs of the weekly checks workflow to simulate a failure and exercise the notification path. This skips the real reusable jobs, adds a dedicated failing job, and updates the notify logic and issue content so test runs create clearly labeled tracking issues instead of waiting for a real scheduled failure.
Use explicit environment variables and a unique multiline output delimiter when collecting changed files, and check out the pull request head repository in the pre-commit job. This makes the formatting workflow more reliable for pull requests, including ones from forks.
C-Achard
added
enhancement
Contributor
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Unresolved CI correctness issues can cause checks to test code other than the intended revision or installed wheel.
Get a fresh assessment by requesting another Copilot review.
Review effort: Balanced
Findings: 2
Open (2)
What changed in this PR
Improves weekly CI reliability, release testing, and failure notifications.
Changes:
- Adds simulated weekly failures and richer tracking issues.
- Removes redundant forced deployment.
- Improves wheel and pull-request workflow handling.
| File | Description |
|---|---|
.github/workflows/weekly_checks.yml |
Adds notification testing and detailed issue reporting. |
.github/workflows/test_and_deploy.yml |
Removes forced deployment and simplifies tag gating. |
.github/workflows/format.yml |
Improves changed-file handling and fork checkout support. |
.github/workflows/build_release.yml |
Separates wheel installation from installed-package tests. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
deruyter92
approved these changes
Sep 21, 2026
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.
Closes #248