Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to Symfony 4.4 #314

Merged
merged 1 commit into from
Feb 23, 2021
Merged

Upgrade to Symfony 4.4 #314

merged 1 commit into from
Feb 23, 2021

Conversation

samwilson
Copy link
Member

@samwilson samwilson commented Feb 23, 2021

  • Updated Symfony packages to 4.4.*.
  • Changed from the deprecated DebugClassLoader.
  • Removed the deprecated WebserverBundle, in favour of the Symfony local server.
  • Removed app_dev.php check for HTTP_X_FORWARDED_FOR header, so it works with the Symfony local server.
  • Removed ./var directory from the repo, to match current Symfony practice. It'll be created at runtime.
  • Increased the minimum version (and platform config) to PHP 7.3, in line with Toolforge.
  • Remove deprecated array_key_exists().
  • Add TransformationFailedException to the EventType data transformer.

Bug: https://phabricator.wikimedia.org/T275463

@samwilson samwilson force-pushed the symfony-4-4 branch 6 times, most recently from 45e9898 to 2c6982c Compare February 23, 2021 05:56
* Updated Symfony packages to `4.4.*`.
* Changed from the deprecated DebugClassLoader.
* Removed the deprecated WebserverBundle, in favour of the Symfony local server.
* Removed app_dev.php check for HTTP_X_FORWARDED_FOR header, so it works with the Symfony local server.
* Removed `./var` directory from the repo, to match current Symfony practice. It'll be created at runtime.
* Increased the minimum version (and platform config) to PHP 7.3, in line with what's in production.
* Remove deprecated `array_key_exists()`.
* Add TransformationFailedException to the EventType data transformer.
* Fix Travis configuration.

Bug: https://phabricator.wikimedia.org/T275463
Copy link
Member

@MusikAnimal MusikAnimal left a comment

Choose a reason for hiding this comment

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

Thank you!!

@MusikAnimal MusikAnimal merged commit 8637b5e into master Feb 23, 2021
@MusikAnimal MusikAnimal deleted the symfony-4-4 branch February 23, 2021 06:36
@MusikAnimal
Copy link
Member

Ah, the README is slightly out of date now (i.e. PHP 7.2). Can be part of the next PR, no rush. Probably don't need to indicate the minimum PHP version anyway or even MySQL as that's implied by composer.json

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants