MokoConsulting/MokoCassiopeia
1
1
Fork 0
Code Issues 2 Pull Requests Actions Packages Releases 1 Activity

Fix revision history tables in documentation and resolve CI workflow errors #35

Merged
Copilot merged 8 commits from copilot/fix-readme-revision-history into main 2026-01-06 08:09:58 +00:00
Conversation 12 Commits 8 Files Changed 3 +8
2 changed files with 8 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 15fa87e49d - Show all commits

2
.github/workflows/joomla_testing.yml vendored
View File

@@ -243,6 +243,8 @@ jobs:
${{ runner.os }}-composer- ${{ runner.os }}-composer-
- name: Install Codeception - name: Install Codeception
env:
COMPOSER_AUTH: '{"github-oauth": {"github.com": "${{ secrets.GITHUB_TOKEN }}"}}'
run: | run: |
copilot-pull-request-reviewer[bot] commented 2026-01-06 08:14:46 +00:00 (Migrated from github.com)
Review
Copy Link

The cache key uses hashFiles('**/composer.lock') but there is no composer.lock file in this repository. This means the hash will always return an empty string, making the cache key effectively ${{ runner.os }}-composer-codeception-8.1-, which is the same as the restore-key. This doesn't break functionality but makes the cache key less specific than intended. Since this is a global composer installation (not project-specific), consider either removing the hashFiles part or using a static cache key.

          key: ${{ runner.os }}-composer-codeception-8.1-global
The cache key uses `hashFiles('**/composer.lock')` but there is no composer.lock file in this repository. This means the hash will always return an empty string, making the cache key effectively `${{ runner.os }}-composer-codeception-8.1-`, which is the same as the restore-key. This doesn't break functionality but makes the cache key less specific than intended. Since this is a global composer installation (not project-specific), consider either removing the hashFiles part or using a static cache key. ```suggestion key: ${{ runner.os }}-composer-codeception-8.1-global ```
composer global require codeception/codeception composer global require codeception/codeception
composer global require codeception/module-db composer global require codeception/module-db

6
.github/workflows/php_quality.yml vendored
View File

@@ -49,6 +49,8 @@ jobs:
${{ runner.os }}-composer- ${{ runner.os }}-composer-
- name: Install PHP_CodeSniffer - name: Install PHP_CodeSniffer
env:
COMPOSER_AUTH: '{"github-oauth": {"github.com": "${{ secrets.GITHUB_TOKEN }}"}}'
run: | run: |
composer global require "squizlabs/php_codesniffer:^3.0" --with-all-dependencies composer global require "squizlabs/php_codesniffer:^3.0" --with-all-dependencies
composer global require "phpcompatibility/php-compatibility:^9.0" --with-all-dependencies composer global require "phpcompatibility/php-compatibility:^9.0" --with-all-dependencies
@@ -103,6 +105,8 @@ jobs:
${{ runner.os }}-composer- ${{ runner.os }}-composer-
- name: Install PHPStan - name: Install PHPStan
env:
COMPOSER_AUTH: '{"github-oauth": {"github.com": "${{ secrets.GITHUB_TOKEN }}"}}'
run: | run: |
composer global require phpstan/phpstan "^1.0" --with-all-dependencies composer global require phpstan/phpstan "^1.0" --with-all-dependencies
composer global require phpstan/extension-installer "^1.0" --with-all-dependencies composer global require phpstan/extension-installer "^1.0" --with-all-dependencies
copilot-pull-request-reviewer[bot] commented 2026-01-06 08:14:45 +00:00 (Migrated from github.com)
Review
Copy Link

The cache key uses hashFiles('**/composer.lock') but there is no composer.lock file in this repository. This means the hash will always return an empty string, making the cache key effectively ${{ runner.os }}-composer-phpstan-${{ matrix.php-version }}-, which is the same as the restore-key. This doesn't break functionality but makes the cache key less specific than intended. Since this is a global composer installation (not project-specific), consider either removing the hashFiles part or using a static cache key.

          key: ${{ runner.os }}-composer-phpstan-${{ matrix.php-version }}
The cache key uses `hashFiles('**/composer.lock')` but there is no composer.lock file in this repository. This means the hash will always return an empty string, making the cache key effectively `${{ runner.os }}-composer-phpstan-${{ matrix.php-version }}-`, which is the same as the restore-key. This doesn't break functionality but makes the cache key less specific than intended. Since this is a global composer installation (not project-specific), consider either removing the hashFiles part or using a static cache key. ```suggestion key: ${{ runner.os }}-composer-phpstan-${{ matrix.php-version }} ```
@@ -150,6 +154,8 @@ jobs:
${{ runner.os }}-composer- ${{ runner.os }}-composer-
- name: Install dependencies - name: Install dependencies
env:
COMPOSER_AUTH: '{"github-oauth": {"github.com": "${{ secrets.GITHUB_TOKEN }}"}}'
run: | run: |
composer global require "squizlabs/php_codesniffer:^3.0" --with-all-dependencies composer global require "squizlabs/php_codesniffer:^3.0" --with-all-dependencies
composer global require "phpcompatibility/php-compatibility:^9.0" --with-all-dependencies composer global require "phpcompatibility/php-compatibility:^9.0" --with-all-dependencies