From e498b22b664c158f3288998f78968cbe08f05cc1 Mon Sep 17 00:00:00 2001 From: Jonathan Miller Date: Sat, 4 Jul 2026 14:22:20 -0500 Subject: [PATCH] ci: skip Joomla metadata check on template repos; fix sample script PHP - pr-metadata-check.yml: guard the validate-metadata job with !startsWith(github.event.repository.name, 'Template-'). Template repos have no real extension manifest, so the check cannot pass by design. - samples/script/*.php: give each installer script a valid class name (ComponentInstallerScript, ModuleInstallerScript, PluginInstallerScript, PackageInstallerScript) instead of the invalid `class {REPONAME}...` placeholder, so the files pass `php -l`. This matters for repos generated from this template that keep samples/. String/comment placeholders remain for find-replace. Authored-by: Moko Consulting --- .mokogitea/workflows/pr-metadata-check.yml | 2 ++ samples/script/script_component.php | 2 +- samples/script/script_module.php | 2 +- samples/script/script_package.php | 2 +- samples/script/script_plugin.php | 2 +- 5 files changed, 6 insertions(+), 4 deletions(-) diff --git a/.mokogitea/workflows/pr-metadata-check.yml b/.mokogitea/workflows/pr-metadata-check.yml index b4c9cbd..5e2be9d 100644 --- a/.mokogitea/workflows/pr-metadata-check.yml +++ b/.mokogitea/workflows/pr-metadata-check.yml @@ -28,6 +28,8 @@ jobs: validate-metadata: name: "Validate Joomla Metadata" runs-on: ubuntu-latest + # Skip on template repos (Template-*) — they have no real extension manifest to validate. + if: ${{ !startsWith(github.event.repository.name, 'Template-') }} steps: - name: Checkout diff --git a/samples/script/script_component.php b/samples/script/script_component.php index d76930f..8a006d4 100644 --- a/samples/script/script_component.php +++ b/samples/script/script_component.php @@ -22,7 +22,7 @@ use Joomla\CMS\Log\Log; defined('_JEXEC') || die; -class {REPONAME}InstallerScript +class ComponentInstallerScript { /** * @var string The saved download key cached during preflight to survive updates diff --git a/samples/script/script_module.php b/samples/script/script_module.php index 19ccc0f..f9ace32 100644 --- a/samples/script/script_module.php +++ b/samples/script/script_module.php @@ -22,7 +22,7 @@ use Joomla\CMS\Log\Log; defined('_JEXEC') || die; -class {REPONAME}InstallerScript +class ModuleInstallerScript { /** * @var string The saved download key cached during preflight to survive updates diff --git a/samples/script/script_package.php b/samples/script/script_package.php index e1ea8e2..12afa1a 100644 --- a/samples/script/script_package.php +++ b/samples/script/script_package.php @@ -22,7 +22,7 @@ use Joomla\CMS\Log\Log; defined('_JEXEC') || die; -class {REPONAME}InstallerScript +class PackageInstallerScript { /** * @var string The saved download key cached during preflight to survive updates diff --git a/samples/script/script_plugin.php b/samples/script/script_plugin.php index e498a1f..a9c4ef5 100644 --- a/samples/script/script_plugin.php +++ b/samples/script/script_plugin.php @@ -22,7 +22,7 @@ use Joomla\CMS\Log\Log; defined('_JEXEC') || die; -class {REPONAME}InstallerScript +class PluginInstallerScript { /** * @var string The saved download key cached during preflight to survive updates