Conversation
- Resolves conflicts: bump uv to 0.11.16, ruff to v0.15.14 - Removes orphaned workflows (rhiza_book, rhiza_ci, rhiza_codeql, rhiza_marimo, rhiza_weekly) - Adds new test files: test_ci_workflow.py, test_release_workflow.py Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the repository’s Rhiza sync configuration and the synced template outputs, including adoption of the new github-project profile, refreshed workflow/tooling versions, and additional validation tests around templates and GitHub workflows.
Changes:
- Bump
.rhiza/.rhiza-versionto0.14.1and refresh synced template artifacts (template.yml,template.lock). - Add configurable GitHub CI OS matrix generation via
make ci-os-matrixand updaterhiza_ci.ymlto consume it. - Expand Rhiza test coverage for workflow structure (CI + release), template bundle/profile validation, and security/pre-commit configuration.
Reviewed changes
Copilot reviewed 16 out of 17 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
.rhiza/tests/structure/test_template_bundles.py |
Adds validation for bundle dependencies and profile references in template-bundles.yml (when present). |
.rhiza/tests/security/test_security_patterns.py |
Adds conditional skipping of GitHub-file requirements based on template configuration; adds a pre-commit hook presence test. |
.rhiza/tests/api/test_release_workflow.py |
New tests asserting structure of rhiza_release.yml, including update-changelog. |
.rhiza/tests/api/test_makefile_targets.py |
Removes tests for the removed coverage-badge target. |
.rhiza/tests/api/test_ci_workflow.py |
New tests for rhiza_ci.yml OS-matrix generation and ci-os-matrix Make target behavior. |
.rhiza/template.yml |
Updates template ref and adds github-project profile; adjusts template selection. |
.rhiza/template.lock |
Updates lock metadata, file list, and adds profiles entry reflecting the new sync state. |
.rhiza/rhiza.mk |
Adds ci-os-matrix target and wires it into .PHONY. |
.rhiza/make.d/test.mk |
Removes the coverage-badge target implementation. |
.rhiza/.rhiza-version |
Bumps Rhiza version to 0.14.1. |
.rhiza/.env |
Adds RHIZA_CI_OS_MATRIX default JSON array for CI. |
.pre-commit-config.yaml |
Adds a local hook to prevent committing Python cache files; bumps tool hook revisions. |
.github/workflows/rhiza_weekly.yml |
Bumps uv version used in the workflow. |
.github/workflows/rhiza_release.yml |
Bumps uv version in existing jobs; adds update-changelog job. |
.github/workflows/rhiza_marimo.yml |
Bumps uv version used in the workflow. |
.github/workflows/rhiza_ci.yml |
Adds generated OS matrix output; retries uv install; bumps uv version. |
.github/workflows/rhiza_book.yml |
Bumps uv version used in the workflow. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| name: Prevent committing Python cache files | ||
| language: fail | ||
| entry: Python cache files (__pycache__, .pyc, .pyo, .pyd) must not be committed. Run 'make clean' to remove them. | ||
| files: '(/__pycache__/|\.py[cod]$)' |
Comment on lines
+22
to
+28
| def _github_bundle_active(repo_root: pathlib.Path | None = None) -> bool: | ||
| """Return True if the github bundle is included in .rhiza/template.yml. | ||
|
|
||
| When running inside the rhiza framework repo itself (no template.yml present), | ||
| github files are always expected to exist. In downstream repos the github | ||
| bundle must be explicitly listed in ``templates`` for these checks to apply. | ||
|
|
| token: ${{ secrets.GITHUB_TOKEN }} | ||
|
|
||
| - name: Install uv | ||
| uses: astral-sh/setup-uv@v8.1.0 |
Member
Author
|
@copilot resolve the merge conflicts in this pull request |
Co-authored-by: tschm <2046079+tschm@users.noreply.github.com>
Contributor
Resolved the merge conflicts by merging Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
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.
Summary
.rhiza-versionfrom0.12.1to0.14.1github-projectprofile totemplate.ymlgithub-teststemplate (now covered by profile)make syncto update template files🤖 Generated with Claude Code