nfeldt/minecraftinator
Watch
1
1
Fork
You've already forked minecraftinator
 0
Code Issues Pull requests 1 Projects Releases Packages Wiki Activity Actions

Move CI to Woodpecker #3

Open
bseger wants to merge 2 commits from ci/woodpecker-v2 into master
pull from: ci/woodpecker-v2
merge into: nfeldt:master
Conversation 0 Commits 2 Files changed 14 +138 -138
bseger commented 2026-09-27 19:47:47 +00:00
Collaborator
Copy link

Replaces the Forgejo Actions workflow with a Woodpecker pipeline; the earlier PRs #1 and #2 were lost from master. The old workflow ran docker build, which the cluster's runners cannot do (no privileged steps, no user namespaces), so the image is assembled with crane from prebuilt binaries, as in the neverup repos.

  • push: generated-files check, golangci-lint, make test, build
  • tag: build manager and gateway, publish git.thehoster.org/nfeldt/minecraftinator:<tag>, also latest

golangci-lint moves to v2.14.0, since v2.1.0 rejects a Go 1.26 module. The second commit fixes the 11 findings it reported; goconst is excluded for test files. Two SSH console error messages changed wording ("the Minecraft server is unavailable" / "is not running").

The e2e suite is not run: it needs kind and Docker.

Needed before this does anything:

  1. Enable the repo in Woodpecker (ci.thstr.io).
  2. Add repo secrets registry_user and registry_token for the tag event; the token needs write:package on the nfeldt namespace.

The check and build steps were run locally in the pipeline's images and pass. publish is untested until a tag is pushed.

🤖 Generated with Claude Code

Replaces the Forgejo Actions workflow with a Woodpecker pipeline; the earlier PRs #1 and #2 were lost from `master`. The old workflow ran `docker build`, which the cluster's runners cannot do (no privileged steps, no user namespaces), so the image is assembled with `crane` from prebuilt binaries, as in the neverup repos. - **push:** generated-files check, `golangci-lint`, `make test`, build - **tag:** build `manager` and `gateway`, publish `git.thehoster.org/nfeldt/minecraftinator:<tag>`, also `latest` golangci-lint moves to v2.14.0, since v2.1.0 rejects a Go 1.26 module. The second commit fixes the 11 findings it reported; `goconst` is excluded for test files. Two SSH console error messages changed wording ("the Minecraft server is unavailable" / "is not running"). The e2e suite is not run: it needs kind and Docker. Needed before this does anything: 1. Enable the repo in Woodpecker (`ci.thstr.io`). 2. Add repo secrets `registry_user` and `registry_token` for the `tag` event; the token needs `write:package` on the `nfeldt` namespace. The check and build steps were run locally in the pipeline's images and pass. `publish` is untested until a tag is pushed. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Move CI to Woodpecker
Some checks failed
ci/woodpecker/push/woodpecker Pipeline failed
Details
6470a223ce
 
The Forgejo workflow ran docker build, which the cluster's runners cannot
do: no privileged steps and no user namespaces. The image is now assembled
with crane from prebuilt binaries, as in the neverup repos.

Adds lint, envtest tests, a generated-files check and a build on push.
Drops the unused Kubebuilder GitHub workflows. golangci-lint moves to
v2.14.0 because v2.1.0 is built with Go 1.24 and rejects a Go 1.26 module.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Fix lint findings
All checks were successful
ci/woodpecker/push/woodpecker Pipeline was successful
Details
ef45e487f7
 
golangci-lint v2.14.0 reported 11 issues. goconst is excluded for test
files, where repeated literals are the readable form.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
All checks were successful
ci/woodpecker/push/woodpecker Pipeline was successful
Details
This pull request can be merged automatically.
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 ci/woodpecker-v2:ci/woodpecker-v2
git switch ci/woodpecker-v2

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch master
git merge --no-ff ci/woodpecker-v2
git switch ci/woodpecker-v2
git rebase master
git switch master
git merge --ff-only ci/woodpecker-v2
git switch ci/woodpecker-v2
git rebase master
git switch master
git merge --no-ff ci/woodpecker-v2
git switch master
git merge --squash ci/woodpecker-v2
git switch master
git merge --ff-only ci/woodpecker-v2
git switch master
git merge ci/woodpecker-v2
git push origin master
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
nfeldt/minecraftinator!3
No description provided.