keel-wordpress: the tests workflow never runs on its default branch #9

New issue

Closed

opened 2026-09-27 17:30:52 -03:00 by marcos-mendez · 1 comment

marcos-mendez commented

2026-09-27 17:30:52 -03:00

(Migrated from github.com)

keel-linux/keel-wordpress has no main branch. Its default branch is master (it is a fork of turnkeylinux-apps/wordpress), and .github/workflows/tests.yml says

on:
  pull_request:
  push:
    branches: [main]

so a push to master runs nothing. Branch protection on master requires
tests / coverage, package / changelog and appliance / build-and-boot, so
pull requests are gated, but the branch itself is never tested after a merge,
and appliance / build-and-boot never runs against what master actually
holds.

Found while closing tracker#7: the appliance job could only be re-run by
re-running the pull request's workflow.

Fix is one line, branches: [master], plus a decision on whether this
repository should be renamed to main like the other recipes. keel-nodebb had
the mirror image of this defect, a required package / changelog with no job
to report it, fixed in keel-nodebb#11.

`keel-linux/keel-wordpress` has no `main` branch. Its default branch is `master` (it is a fork of `turnkeylinux-apps/wordpress`), and `.github/workflows/tests.yml` says on: pull_request: push: branches: [main] so a push to `master` runs nothing. Branch protection on `master` requires `tests / coverage`, `package / changelog` and `appliance / build-and-boot`, so pull requests are gated, but the branch itself is never tested after a merge, and `appliance / build-and-boot` never runs against what `master` actually holds. Found while closing tracker#7: the `appliance` job could only be re-run by re-running the pull request's workflow. Fix is one line, `branches: [master]`, plus a decision on whether this repository should be renamed to `main` like the other recipes. keel-nodebb had the mirror image of this defect, a required `package / changelog` with no job to report it, fixed in keel-nodebb#11.

marcos-mendez commented

2026-09-29 17:37:26 -03:00

(Migrated from github.com)

Fixed: .github/workflows/tests.yml in keel-wordpress now runs on push to master, its default branch.

Fixed: `.github/workflows/tests.yml` in keel-wordpress now runs on `push` to `master`, its default branch.