diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 8d52769..8cbaaa0 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -1,195 +1,195 @@ name: Continuous Integration Pipeline on: - push: - branches: - - main - - "version/*" - pull_request: - branches: - - main + push: + branches: + - main + - "version/*" + pull_request: + branches: + - main permissions: - contents: read + contents: read jobs: - formatting-fixes: - name: Run formatting fix scripts - runs-on: ubuntu-latest + formatting-fixes: + name: Run formatting fix scripts + runs-on: ubuntu-latest - steps: - - name: Check out repository - uses: actions/checkout@v4 - with: - fetch-depth: 0 + steps: + - name: Check out repository + uses: actions/checkout@v4 + with: + fetch-depth: 0 - - name: Ensure script executability - shell: bash - run: | - set -euo pipefail - chmod +x scripts/*.sh 2>/dev/null || true + - name: Ensure script executability + shell: bash + run: | + set -euo pipefail + chmod +x scripts/*.sh 2>/dev/null || true - - name: Run fix_tabs.sh if present - shell: bash - run: | - set -euo pipefail - cd "${GITHUB_WORKSPACE}" - if [ -f "scripts/fix_tabs.sh" ]; then - echo "Running scripts/fix_tabs.sh" - bash scripts/fix_tabs.sh - else - echo "scripts/fix_tabs.sh not found. Skipping." - fi + - name: Run fix_tabs.sh if present + shell: bash + run: | + set -euo pipefail + cd "${GITHUB_WORKSPACE}" + if [ -f "scripts/fix_tabs.sh" ]; then + echo "Running scripts/fix_tabs.sh" + bash scripts/fix_tabs.sh + else + echo "scripts/fix_tabs.sh not found. Skipping." + fi - - name: Run fix_paths.sh if present - shell: bash - run: | - set -euo pipefail - cd "${GITHUB_WORKSPACE}" - if [ -f "scripts/fix_paths.sh" ]; then - echo "Running scripts/fix_paths.sh" - bash scripts/fix_paths.sh - else - echo "scripts/fix_paths.sh not found. Skipping." - fi + - name: Run fix_paths.sh if present + shell: bash + run: | + set -euo pipefail + cd "${GITHUB_WORKSPACE}" + if [ -f "scripts/fix_paths.sh" ]; then + echo "Running scripts/fix_paths.sh" + bash scripts/fix_paths.sh + else + echo "scripts/fix_paths.sh not found. Skipping." + fi - - name: Fail if formatting scripts modified files - shell: bash - run: | - set -euo pipefail - if ! git diff --quiet; then - echo "Formatting scripts introduced changes." - echo "Run fix_tabs.sh and fix_paths.sh locally and commit the results." - git diff - exit 1 - fi + - name: Fail if formatting scripts modified files + shell: bash + run: | + set -euo pipefail + if ! git diff --quiet; then + echo "Formatting scripts introduced changes." + echo "Run fix_tabs.sh and fix_paths.sh locally and commit the results." + git diff + exit 1 + fi - php-lint: - name: PHP lint - runs-on: ubuntu-latest - needs: formatting-fixes + php-lint: + name: PHP lint + runs-on: ubuntu-latest + needs: formatting-fixes - steps: - - name: Check out repository - uses: actions/checkout@v4 + steps: + - name: Check out repository + uses: actions/checkout@v4 - - name: Set up PHP - uses: shivammathur/setup-php@v2 - with: - php-version: "8.2" - coverage: none + - name: Set up PHP + uses: shivammathur/setup-php@v2 + with: + php-version: "8.2" + coverage: none - - name: Lint PHP files under src - shell: bash - run: | - set -euo pipefail - if [ -d "src" ]; then - echo "Running php -l against PHP files in src/" - find src -type f -name "*.php" -print0 | xargs -0 -n 1 -P 4 php -l - else - echo "No src directory found. Skipping PHP lint." - fi + - name: Lint PHP files under src + shell: bash + run: | + set -euo pipefail + if [ -d "src" ]; then + echo "Running php -l against PHP files in src/" + find src -type f -name "*.php" -print0 | xargs -0 -n 1 -P 4 php -l + else + echo "No src directory found. Skipping PHP lint." + fi - composer-tests: - name: Composer install and tests - runs-on: ubuntu-latest - needs: php-lint + composer-tests: + name: Composer install and tests + runs-on: ubuntu-latest + needs: php-lint - steps: - - name: Check out repository - uses: actions/checkout@v4 + steps: + - name: Check out repository + uses: actions/checkout@v4 - - name: Set up PHP - uses: shivammathur/setup-php@v2 - with: - php-version: "8.2" - coverage: none + - name: Set up PHP + uses: shivammathur/setup-php@v2 + with: + php-version: "8.2" + coverage: none - - name: Install dependencies if composer.json exists - shell: bash - run: | - set -euo pipefail - if [ -f "composer.json" ]; then - composer install --no-interaction --no-progress --prefer-dist - else - echo "No composer.json found. Skipping dependency install and tests." - fi + - name: Install dependencies if composer.json exists + shell: bash + run: | + set -euo pipefail + if [ -f "composer.json" ]; then + composer install --no-interaction --no-progress --prefer-dist + else + echo "No composer.json found. Skipping dependency install and tests." + fi - - name: Run Composer tests when defined - shell: bash - run: | - set -euo pipefail - if [ ! -f "composer.json" ]; then - echo "No composer.json. Nothing to test." - exit 0 - fi + - name: Run Composer tests when defined + shell: bash + run: | + set -euo pipefail + if [ ! -f "composer.json" ]; then + echo "No composer.json. Nothing to test." + exit 0 + fi - if composer run -q | grep -q "^ test"; then - echo "Detected composer script 'test'. Running composer test." - composer test - else - echo "No 'test' script defined in composer.json. Skipping tests." - fi + if composer run -q | grep -q "^ test"; then + echo "Detected composer script 'test'. Running composer test." + composer test + else + echo "No 'test' script defined in composer.json. Skipping tests." + fi - validate-manifest-and-changelog: - name: Validate manifest and changelog via scripts - runs-on: ubuntu-latest - needs: - - php-lint - - composer-tests + validate-manifest-and-changelog: + name: Validate manifest and changelog via scripts + runs-on: ubuntu-latest + needs: + - php-lint + - composer-tests - steps: - - name: Check out repository - uses: actions/checkout@v4 - with: - fetch-depth: 0 + steps: + - name: Check out repository + uses: actions/checkout@v4 + with: + fetch-depth: 0 - - name: Ensure script executability - shell: bash - run: | - set -euo pipefail - chmod +x scripts/*.sh 2>/dev/null || true + - name: Ensure script executability + shell: bash + run: | + set -euo pipefail + chmod +x scripts/*.sh 2>/dev/null || true - - name: Run manifest validation script if present - shell: bash - run: | - set -euo pipefail - echo "Checking for manifest validation scripts" + - name: Run manifest validation script if present + shell: bash + run: | + set -euo pipefail + echo "Checking for manifest validation scripts" - if [ -f "scripts/validate_manifest.sh" ]; then - echo "Running scripts/validate_manifest.sh" - bash scripts/validate_manifest.sh - elif [ -f "scripts/validate_manifest.php" ]; then - echo "Running scripts/validate_manifest.php" - php scripts/validate_manifest.php - else - echo "No manifest validation script found (scripts/validate_manifest.sh or scripts/validate_manifest.php). Skipping manifest validation step." - fi + if [ -f "scripts/validate_manifest.sh" ]; then + echo "Running scripts/validate_manifest.sh" + bash scripts/validate_manifest.sh + elif [ -f "scripts/validate_manifest.php" ]; then + echo "Running scripts/validate_manifest.php" + php scripts/validate_manifest.php + else + echo "No manifest validation script found (scripts/validate_manifest.sh or scripts/validate_manifest.php). Skipping manifest validation step." + fi - - name: Run changelog update or verification script if present - shell: bash - run: | - set -euo pipefail - echo "Checking for changelog update or verification scripts" + - name: Run changelog update or verification script if present + shell: bash + run: | + set -euo pipefail + echo "Checking for changelog update or verification scripts" - if [ -f "scripts/update_changelog.sh" ]; then - echo "Running scripts/update_changelog.sh --ci" - bash scripts/update_changelog.sh --ci - elif [ -f "scripts/verify_changelog.sh" ]; then - echo "Running scripts/verify_changelog.sh" - bash scripts/verify_changelog.sh - elif [ -f "scripts/verify_changelog.php" ]; then - echo "Running scripts/verify_changelog.php" - php scripts/verify_changelog.php - else - echo "No changelog script found (scripts/update_changelog.sh, scripts/verify_changelog.sh, or scripts/verify_changelog.php). Skipping changelog enforcement." - exit 0 - fi + if [ -f "scripts/update_changelog.sh" ]; then + echo "Running scripts/update_changelog.sh --ci" + bash scripts/update_changelog.sh --ci + elif [ -f "scripts/verify_changelog.sh" ]; then + echo "Running scripts/verify_changelog.sh" + bash scripts/verify_changelog.sh + elif [ -f "scripts/verify_changelog.php" ]; then + echo "Running scripts/verify_changelog.sh" + php scripts/verify_changelog.php + else + echo "No changelog script found (scripts/update_changelog.sh, scripts/verify_changelog.sh, or scripts/verify_changelog.php). Skipping changelog enforcement." + exit 0 + fi - echo "Checking for uncommitted changes after changelog script" - if ! git diff --quiet; then - echo "Changelog or related files were modified by the script." - echo "Please run the changelog script locally and commit the changes before pushing." - git diff - exit 1 - fi + echo "Checking for uncommitted changes after changelog script" + if ! git diff --quiet; then + echo "Changelog or related files were modified by the script." + echo "Please run the changelog script locally and commit the changes before pushing." + git diff + exit 1 + fi