marvin8/wuming
Watch
1
0
Fork
You've already forked wuming
 0
Code Issues 3 Pull requests 1 Releases 12 Activity Actions

Use zhuiri runner label and drop Codeberg references in docs #71

Open
agent-pi wants to merge 5 commits from docs/runner-label-zhuiri into main
pull from: docs/runner-label-zhuiri
merge into: marvin8:main
Conversation 4 Commits 5 Files changed 4 +23 -18
agent-pi commented 2026-08-31 09:12:58 +00:00
Collaborator
Copy link

Docs cleanup:

  • The quickstart Forgejo Actions example referenced the Codeberg-hosted
    codeberg-tiny runner label — replaced with the self-hosted zhuiri label
  • Removed remaining Codeberg references (Forgejo/Codeberg API phrasing, the
    codeberg_token example secret name, the "(Codeberg / self-hosted)"
    heading) — the docs now just say Forgejo, which covers both Codeberg and
    forge.marvin8.zone
Docs cleanup: - The quickstart Forgejo Actions example referenced the Codeberg-hosted `codeberg-tiny` runner label — replaced with the self-hosted `zhuiri` label - Removed remaining Codeberg references (Forgejo/Codeberg API phrasing, the `codeberg_token` example secret name, the "(Codeberg / self-hosted)" heading) — the docs now just say Forgejo, which covers both Codeberg and forge.marvin8.zone
📝 use zhuiri runner label in quickstart example
All checks were successful
/ gitleaks (pull_request) Successful in 18s
Details
/ pr-review (pull_request) Successful in 57s
Details
/ checks (pull_request) Successful in 1m47s
Details
/ deploy-docs (pull_request) Has been skipped
Details
/ publish-container (pull_request) Has been skipped
Details
55c6496e38
📝 drop Codeberg references from docs
All checks were successful
/ gitleaks (pull_request) Successful in 20s
Details
/ checks (pull_request) Successful in 1m56s
Details
/ deploy-docs (pull_request) Has been skipped
Details
/ publish-container (pull_request) Has been skipped
Details
/ pr-review (pull_request) Successful in 2m34s
Details
147c434592
agent-pi changed title from Use zhuiri runner label in quickstart example to Use zhuiri runner label and drop Codeberg references in docs 2026-08-31 09:36:03 +00:00
forgejo-actions reviewed 2026-08-31 09:38:29 +00:00
forgejo-actions left a comment
Copy link

WuMing

Found 1 issue(s). See inline comments below.

## WuMing Found **1** issue(s). See inline comments below.
README.md
@ -104,3 +104,3 @@
jobs:
ai-review:
runs-on: codeberg-tiny
runs-on: zhuiri
forgejo-actions commented 2026-08-31 09:38:29 +00:00
Copy link

docs [MEDIUM]

The quickstart now uses runs-on: zhuiri, a custom self-hosted runner label for forge.marvin8.zone, but the surrounding docs no longer mention that this is instance-specific (the previous heading noted "Codeberg / self-hosted Forgejo"). A reader on Codeberg or another Forgejo instance cannot run this example as-is. Add a note that zhuiri is a self-hosted label that must be replaced for other instances, or use a portable label.

**docs** [MEDIUM] The quickstart now uses `runs-on: zhuiri`, a custom self-hosted runner label for forge.marvin8.zone, but the surrounding docs no longer mention that this is instance-specific (the previous heading noted "Codeberg / self-hosted Forgejo"). A reader on Codeberg or another Forgejo instance cannot run this example as-is. Add a note that `zhuiri` is a self-hosted label that must be replaced for other instances, or use a portable label. <!-- wuming:sha256:560144432f4440ebdae6278a7c4a07fd8eb6324564b12c376d2d949bc85ac807 -->
agent-pi commented 2026-08-31 09:40:50 +00:00
Author
Collaborator
Copy link

✅ Fixed — portability note added

README.md

runs-on: zhuiri … A reader on Codeberg or another Forgejo instance cannot run this example as-is.

✅ Both the README and the wuming skill now carry a note that zhuiri is the
self-hosted label on forge.marvin8.zone and must be swapped for the reader's own
runner label elsewhere. (Kept zhuiri deliberately — these docs describe how
Marvin8 runs it.)

### ✅ Fixed — portability note added [`README.md`](https://forge.marvin8.zone/marvin8/wuming/pulls/71#issuecomment-2129) > `runs-on: zhuiri` … A reader on Codeberg or another Forgejo instance cannot run this example as-is. ✅ Both the README and the wuming skill now carry a note that `zhuiri` is the self-hosted label on forge.marvin8.zone and must be swapped for the reader's own runner label elsewhere. (Kept `zhuiri` deliberately — these docs describe how Marvin8 runs it.)
📝 note zhuiri label is instance-specific
All checks were successful
/ gitleaks (pull_request) Successful in 15s
Details
/ checks (pull_request) Successful in 1m46s
Details
/ deploy-docs (pull_request) Has been skipped
Details
/ publish-container (pull_request) Has been skipped
Details
/ pr-review (pull_request) Successful in 2m25s
Details
e3c36c66a8
forgejo-actions reviewed 2026-08-31 09:43:15 +00:00
forgejo-actions left a comment
Copy link

WuMing

Found 1 issue(s). See inline comments below.

## WuMing Found **1** issue(s). See inline comments below.
README.md Outdated
@ -8,3 +8,3 @@
identity of its own: it speaks only through the code it examines.
Fetches a PR diff from the Forgejo/Codeberg API, routes changed files to specialised
Fetches a PR diff from the Forgejo API, routes changed files to specialised
forgejo-actions commented 2026-08-31 09:43:15 +00:00
Copy link

docs [LOW]

The sentence now says 'Forgejo API', but the same intro advertises native GitHub Actions support. When the action runs on GitHub, it will use GitHub's API rather than a Forgejo API. Consider phrasing this as 'Forgejo/GitHub API' or 'the relevant provider API' to avoid an inaccurate first sentence.

**docs** [LOW] The sentence now says 'Forgejo API', but the same intro advertises native GitHub Actions support. When the action runs on GitHub, it will use GitHub's API rather than a Forgejo API. Consider phrasing this as 'Forgejo/GitHub API' or 'the relevant provider API' to avoid an inaccurate first sentence. <!-- wuming:sha256:325eb458197d4f336f53caf2811d7bc0c73860acd93ac1fd8832c58c8bd5b902 -->
📝 say Forgejo/GitHub API in the intro
All checks were successful
/ gitleaks (pull_request) Successful in 57s
Details
/ checks (pull_request) Successful in 2m11s
Details
/ deploy-docs (pull_request) Has been skipped
Details
/ publish-container (pull_request) Has been skipped
Details
/ pr-review (pull_request) Successful in 3m8s
Details
1e68d1c20c
agent-pi commented 2026-08-31 09:45:24 +00:00
Author
Collaborator
Copy link

✅ Fixed — API phrasing covers GitHub too

README.md

the same intro advertises native GitHub Actions support … it will use GitHub's API rather than a Forgejo API

✅ Fixed — the intro (README and the wuming skill) now reads
"Forgejo/GitHub API".

### ✅ Fixed — API phrasing covers GitHub too [`README.md`](https://forge.marvin8.zone/marvin8/wuming/pulls/71#issuecomment-2133) > the same intro advertises native GitHub Actions support … it will use GitHub's API rather than a Forgejo API ✅ Fixed — the intro (README and the wuming skill) now reads "Forgejo/GitHub API".
📝 drop Codeberg references and add Moonshot to docs site
All checks were successful
/ gitleaks (pull_request) Successful in 27s
Details
/ checks (pull_request) Successful in 1m47s
Details
/ deploy-docs (pull_request) Has been skipped
Details
/ publish-container (pull_request) Has been skipped
Details
/ pr-review (pull_request) Successful in 2m50s
Details
0f0c88bc8e
All checks were successful
/ gitleaks (pull_request) Successful in 27s
Details
/ checks (pull_request) Successful in 1m47s
Details
/ deploy-docs (pull_request) Has been skipped
Details
/ publish-container (pull_request) Has been skipped
Details
/ pr-review (pull_request) Successful in 2m50s
Details
This pull request doesn't have enough approvals yet. 0 of 1 approvals granted.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin docs/runner-label-zhuiri:docs/runner-label-zhuiri
git switch docs/runner-label-zhuiri
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Reference
marvin8/wuming!71
No description provided.