Skip to content

Disable inherited GrumPHP autoload validation for fixture extensions - #137

Merged
Kanti merged 2 commits into
mainfrom
copilot/fix-github-actions-job-php-8-2-typo3-13
Sep 25, 2026
Merged

Kanti merged 2 commits into
mainfrom
copilot/fix-github-actions-job-php-8-2-typo3-13

Conversation

Copilot AI commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

The php: 8.2 TYPO3: 13 Actions job was failing in GrumPHP’s composer_validate_autoload task. The failure came from functional test fixture classes under Tests/Functional/Fixtures/Extensions/blog_example, which are intentionally not part of this extension’s PSR-4 autoload mapping.

  • Root cause

    • grumphp.yml imports pluswerk/grumphp-config, which enables composer_validate_autoload by default.
    • That task validates the repository against the extension’s Composer autoload rules and flags the fixture extension classes as violations.
  • Change

    • Add a local override in grumphp.yml to disable only the inherited composer_validate_autoload task.
    • Keep the rest of the shared GrumPHP configuration unchanged.
  • Why this scope

    • The failing classes live in functional test fixtures, not in the extension’s production autoload tree.
    • This avoids changing fixture namespaces or broader lint behavior for unrelated tasks.
grumphp:
  tasks:
    composer_validate_autoload:
      metadata:
        enabled: false

Co-authored-by: Kanti <471387+Kanti@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job for PHP 8.2 and TYPO3 13 Disable inherited GrumPHP autoload validation for fixture extensions Sep 25, 2026
Copilot AI requested a review from Kanti September 25, 2026 07:43
@Kanti
Kanti marked this pull request as ready for review September 25, 2026 07:44
@Kanti
Kanti merged commit b2519d1 into main Sep 25, 2026
33 checks passed
@Kanti
Kanti deleted the copilot/fix-github-actions-job-php-8-2-typo3-13 branch September 25, 2026 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants