chore: add the github-paper bundle - #907
Conversation
|
Warning Review limit reached
Next review available in: 52 minutes Limit details: You’ve used the included review currently available. 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?Wait for the limit to reset, then comment An organization admin can change what happens after included review limits in Billing. How do review limits work?CodeRabbit enforces per-developer PR review limits within each organization. For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
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. Comment |
There was a problem hiding this comment.
🟡 Changes recommended
The newly added paper README’s entry-point selection logic does not match the pinned reusable workflow behavior, and the workflow stub includes a misleading/redundant secrets: inherit.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Adds the Rhiza github-paper bundle so the repository gains a GitHub Actions workflow to compile a future LaTeX paper under docs/paper/ and includes bundled documentation describing the expected paper folder layout.
Changes:
- Add
.github/workflows/rhiza_paper.ymlstub to run Rhiza’s reusable “paper” workflow ondocs/paper/**changes. - Add
docs/paper/README.mddescribing the intended LaTeX paper structure and build targets. - Update Rhiza template configuration/lock to include the
github-papertemplate.
File summaries
| File | Description |
|---|---|
docs/paper/README.md |
Adds documentation for where/how to place LaTeX sources and how the paper build is expected to work. |
.rhiza/template.yml |
Enables the github-paper template bundle in the Rhiza sync configuration. |
.rhiza/template.lock |
Records the synced template state including the newly added workflow and docs file. |
.github/workflows/rhiza_paper.yml |
Adds a workflow stub that delegates paper compilation to Rhiza’s reusable workflow. |
Review details
- Files reviewed: 3/4 changed files
- Comments generated: 3
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| The root document is chosen by name, in this order: | ||
|
|
||
| 1. `main.tex` | ||
| 2. `paper.tex` | ||
| 3. the first `.tex` file alphabetically |
| A folder holding one `.tex` file is unambiguous whatever it is called, which is the | ||
| common case. Name it `main.tex` if you have several and want to be explicit. |
| uses: jebel-quant/rhiza/.github/workflows/rhiza_paper.yml@v1.5.0 | ||
| secrets: inherit | ||
| # `contents: read` only. The `write` scope this stub used to grant existed solely for |
Adds the
github-paperbundle to.rhiza/template.ymland applies the sync at thealready-pinned template ref
v1.5.0(no version bump — that stays a separate change).What the bundle brings in:
.github/workflows/rhiza_paper.yml— compilesdocs/paper/*.texwithlatexmkanduploads the PDF as a workflow artifact. It only triggers on changes under
docs/paper/**(or to the workflow itself), so it costs nothing until there is apaper to build.
docs/paper/README.md— from thepaperbundle (arequires:ofgithub-paper),documenting the layout the workflow expects: root document
main.tex, withchapters/figures/bib in subdirectories.
No
.texsource is added, so nothing builds yet: dropdocs/paper/main.texin and theworkflow starts running. The durable copy of the PDF comes from the book (the paper
folder sits inside the docs tree), not from the 30-day artifact.
Files changed by the sync: 2 added, plus
.rhiza/template.lock. Nothing was leftunstaged, no conflicts. No gates were run — run
/rhiza:qualityfor a scorecard.