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

Provide PHP 8.1 support #74

Merged
merged 1 commit into from
Dec 1, 2021

Conversation

weierophinney
Copy link
Contributor

This patch provides PHP 8.1 support, via the following changes:

  • Adds ~8.1.0 to the list of allowed PHP versions.
  • Marks a conflict with symfony/console versions prior to 5.3.11 (first version supporting PHP 8.1)
  • Replaces usage of gmstrftime() with a combination of casting timestamps to DateTimeImmutable objects with a GMT timezone, and using IntlDateFormatter::formatObject() with an equivalent date/time format to what was used with gmstrftime().
  • Adds #[\ReturnTimeWillChange] attributes to classes implementing JsonSerialize to ensure they remain both backwards and forwards compatible.

@weierophinney weierophinney added this to the 3.5.0 milestone Dec 1, 2021
@weierophinney weierophinney added the Enhancement New feature or request label Dec 1, 2021
This patch provides PHP 8.1 support, via the following changes:

- Adds `~8.1.0` to the list of allowed PHP versions.
- Marks a conflict with symfony/console versions prior to 5.3.11 (first version supporting PHP 8.1)
- Replaces usage of `gmstrftime()` with a combination of casting timestamps to `DateTimeImmutable` objects with a GMT timezone, and using `IntlDateFormatter::formatObject()` with an equivalent date/time format to what was used with `gmstrftime()`.
- Adds `#[\ReturnTimeWillChange]` attributes to classes implementing `JsonSerialize` to ensure they remain both backwards and forwards compatible.

Signed-off-by: Matthew Weier O'Phinney <matthew@weierophinney.net>
@Ocramius
Copy link
Member

Ocramius commented Dec 1, 2021

If you can release mezzio/mezzio#96 (and agree with it), we can probably even skip the --ignore-platform-reqs ballet :D

@weierophinney
Copy link
Contributor Author

If you can release mezzio/mezzio#96 (and agree with it), we can probably even skip the --ignore-platform-reqs ballet :D

There's enough third-party requirements that my guess is we'd still have to. :)

@weierophinney
Copy link
Contributor Author

@Ocramius laminas-coding-standard isn't compatible yet, which is the blocker for skipping --ignore-platform-reqs.

@Ocramius Ocramius merged commit e72d8c1 into mezzio:3.5.x Dec 1, 2021
@weierophinney weierophinney deleted the feature/php-8.1-support branch December 1, 2021 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants