Skip to content

Bump symfony/polyfill versions for grav 1.8#4030

Open
Rotzbua wants to merge 1 commit intogetgrav:1.8from
Rotzbua:feat_update_polyfill-php_1_8
Open

Bump symfony/polyfill versions for grav 1.8#4030
Rotzbua wants to merge 1 commit intogetgrav:1.8from
Rotzbua:feat_update_polyfill-php_1_8

Conversation

@Rotzbua
Copy link
Contributor

@Rotzbua Rotzbua commented Feb 12, 2026

Changes

  • install newer polyfills
  • do not install old polyfills
  • remove unnecessary polyfills

Todo

Update composer.lock (should be done by maintainer because it is hard to review a lock file)

Reference

Mentioned in #4031

Copilot AI review requested due to automatic review settings February 12, 2026 18:02
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates Grav’s Composer dependency metadata to align Symfony polyfill usage with the PHP 8.3+ baseline targeted for Grav 1.8.

Changes:

  • Bump symfony/polyfill-mbstring and symfony/polyfill-iconv to ^1.33.
  • Switch PHP polyfill requirements to symfony/polyfill-php84 (and add symfony/polyfill-php85).
  • Update the replace section to replace symfony/polyfill-php80 through symfony/polyfill-php83.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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