Skip to content

[!!!][TASK] Validate PHP 8.5 with TYPO3 13.4 - #8

Open
N0S0I wants to merge 5 commits into
mainfrom
php85
Open

N0S0I wants to merge 5 commits into
mainfrom
php85

Conversation

@N0S0I

@N0S0I N0S0I commented Oct 23, 2026

Copy link
Copy Markdown

Restrict Composer, extension metadata and CI to TYPO3 13.4, and retain the existing static checks. Add small configuration type annotations and update the DDEV PHP version hint.

Breaking change: legacy TYPO3 support is removed; the new development branch requires TYPO3 13.4.

Validation: the GitHub Actions branch workflow passed for 99d3650d51b9f6c88147cf98d5771e8c9304962f. This PR targets main.

@N0S0I
N0S0I requested a review from weakbit October 23, 2026 06:49

N0S0I commented Oct 23, 2026

Copy link
Copy Markdown
Author

Kanti's requirements from the previous review:

Comment thread .github/workflows/tasks.yml Outdated
Comment on lines +26 to +30
- run: composer require "enshrined/svg-sanitize:~1.0.0" --dev --no-update --sort-packages --no-interaction
- run: composer require typo3/minimal="^${{ matrix.typo3 }}" --sort-packages --no-scripts -W --dev
- run: composer install --no-interaction --no-progress
- name: Run all static checks
run: ./vendor/bin/grumphp run --no-ansi

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- run: composer require "enshrined/svg-sanitize:~1.0.0" --dev --no-update --sort-packages --no-interaction
- run: composer require typo3/minimal="^${{ matrix.typo3 }}" --sort-packages --no-scripts -W --dev
- run: composer install --no-interaction --no-progress
- name: Run all static checks
run: ./vendor/bin/grumphp run --no-ansi
- run: composer require typo3/minimal="^${{ matrix.typo3 }}" --dev --ignore-platform-req=php+
- run: composer install --no-interaction --no-progress --ignore-platform-req=php+
- run: ./vendor/bin/grumphp run --ansi

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