Skip to content

chore: sync GitHub repo metadata from rhiza v1.3.3 - #28

Closed
tschm wants to merge 1 commit into
masterfrom
rhiza_v1.3.3_meta
Closed

chore: sync GitHub repo metadata from rhiza v1.3.3#28
tschm wants to merge 1 commit into
masterfrom
rhiza_v1.3.3_meta

Conversation

@tschm

@tschm tschm commented Aug 17, 2026

Copy link
Copy Markdown
Owner

The part of the jebel-quant/rhiza v0.18.8v1.3.3 sync that nothing at build or test time reads, split out of #27 so it can merge on its own review.

Files (7)

File
.github/CONFIG.md new
.github/DISCUSSION_TEMPLATE/help-wanted.yml new
.github/DISCUSSION_TEMPLATE/ideas.yml new
.github/rulesets/main-branch-protection.json new
.github/rulesets/tag-protection.json new
.github/dependabot.yml updated
.github/pull_request_template.md updated

Why these and nothing else

The remaining 72 files of the sync are coupled and stay together in #27:

  • every rhiza_*.yml workflow is a thin caller pinned at @v1.3.3, and those reusable workflows run against this repo's .rhiza/ plumbing;
  • quality.mk's rhiza-test provisions dependencies for the flat .rhiza/tests layout, and python.mk reads .rhiza/tests/test_pyproject.py;
  • pytest.ini adds pythonpath = .rhiza/tests, and the deleted .rhiza/requirements/*.txt were what the old bootstrap.mk installed from.

Land either order — there is no dependency between this PR and #27.

Note

.rhiza/template.lock lives in #27 and lists these seven files as template-owned, so /rhiza:status will report drift on them until both PRs have merged.

No gates were run — run /rhiza:quality for a scorecard.

The seven files here are the part of the v0.18.8 -> v1.3.3 template sync
that no runtime depends on: issue/discussion templates, branch and tag
rulesets, the dependabot config and the PR template. Nothing reads them at
build or test time, so they carry none of the coupling that keeps the rest
of the sync atomic — the workflow pins, the make.d plumbing and the
.rhiza/tests layout all move in lockstep and stay in the larger PR.

Split out so this part can merge on its own review.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings August 17, 2026 19:18
@coderabbitai

coderabbitai Bot commented Aug 17, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@tschm, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 59 minutes

Limit details: You’ve used all 1 included review currently available under your plan.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 6668d10a-e04b-4dd4-bf58-69c5749c31ca

📥 Commits

Reviewing files that changed from the base of the PR and between 2e3ebf7 and fac3ad8.

📒 Files selected for processing (7)
  • .github/CONFIG.md
  • .github/DISCUSSION_TEMPLATE/help-wanted.yml
  • .github/DISCUSSION_TEMPLATE/ideas.yml
  • .github/dependabot.yml
  • .github/pull_request_template.md
  • .github/rulesets/main-branch-protection.json
  • .github/rulesets/tag-protection.json

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR syncs GitHub repository metadata files from the Rhiza template (v1.3.3), adding standard repo configuration/documentation and aligning templates/rulesets with upstream so maintenance and automation expectations are consistent.

Changes:

  • Add GitHub ruleset JSON definitions for default-branch and version-tag protection.
  • Add GitHub Discussion templates and a .github/CONFIG.md doc describing required/optional Actions secrets.
  • Update dependabot.yml formatting and extend the PR checklist.

Reviewed changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
.github/CONFIG.md Adds documentation for Rhiza workflow secrets and setup.
.github/DISCUSSION_TEMPLATE/help-wanted.yml Adds a “Help Wanted” discussion template.
.github/DISCUSSION_TEMPLATE/ideas.yml Adds an “Idea” discussion template.
.github/rulesets/main-branch-protection.json Adds a default-branch protection ruleset definition.
.github/rulesets/tag-protection.json Adds a version tag protection ruleset definition.
.github/dependabot.yml Minor formatting cleanup.
.github/pull_request_template.md Updates PR checklist (including changelog and dependency check item).

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines +11 to +16
"rules": [
{ "type": "creation" },
{ "type": "update" },
{ "type": "deletion" },
{ "type": "non_fast_forward" }
],
- [ ] `CHANGELOG.md` entry added (or not needed for this change)
- [ ] Documentation updated if behaviour changed
- [ ] `make deptry` passes (no unused or missing dependencies)
- [ ] `make deps` passes (no unused or missing dependencies)
@tschm

tschm commented Aug 17, 2026

Copy link
Copy Markdown
Owner Author

Superseded by alihaskar#91 — these seven files are rhiza-managed repo metadata, so they belong in the parent repo rather than in this fork. Same branch, same commit, retargeted at alihaskar/pycharting:master. Closing here.

@tschm tschm closed this Aug 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants