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

Add PHP8 support #6

Closed

Conversation

jseparovic1
Copy link

@jseparovic1 jseparovic1 commented Oct 19, 2020

Q A
Documentation no
Bugfix no
BC Break no
New Feature yes
RFC no
QA no

Description

Closes #5

  • Modify composer.json to provide support for PHP 8.0 by adding the constraint ~8.0.0
  • Modify composer.json to drop support for PHP less than 7.3
  • Modify composer.json to implement phpunit 9.3 which supports PHP 7.3+
  • Modify .travis.yml to ignore platform requirements when installing composer dependencies (simply add --ignore-platform-reqs to COMPOSER_ARGS env variable)
  • Modify .travis.yml to add PHP 8.0 to the matrix (NOTE: Do not allow failures as PHP 8.0 has a feature freeze since 2020-08-04!)

@jseparovic1 jseparovic1 marked this pull request as ready for review October 19, 2020 21:23
Signed-off-by: Jurica Šeparović <jurica.separovic@gmail.com>
@jseparovic1 jseparovic1 changed the title Add PHP support Add PHP8 support Oct 20, 2020
@froschdesign froschdesign added this to To do in PHP 8.0 via automation Oct 25, 2020
@froschdesign froschdesign added Enhancement New feature or request hacktoberfest-accepted Issues/Pull-Requests which can be fixed during Hacktoberfest: https://hacktoberfest.digitalocean.com labels Oct 25, 2020
PHP 8.0 automation moved this from To do to In progress Oct 25, 2020
phpunit.xml.dist Show resolved Hide resolved
env:
- DEPS=latest
- php: 7.4
- php: nightly
Copy link
Member

Choose a reason for hiding this comment

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

This should be 8.0 now.

env:
- DEPS=lowest
- php: 7.4
- COMPOSER_ARGS="--no-interaction --ignore-platform-reqs"
- php: nightly
Copy link
Member

Choose a reason for hiding this comment

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

This should be 8.0 now.

env:
- DEPS=lowest
- php: 7.4
- COMPOSER_ARGS="--no-interaction --ignore-platform-reqs"
Copy link
Member

Choose a reason for hiding this comment

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

Please remove this line.

env:
- DEPS=latest
- COMPOSER_ARGS="--no-interaction --ignore-platform-reqs"
Copy link
Member

Choose a reason for hiding this comment

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

Please remove this line.

@froschdesign
Copy link
Member

@jseparovic1
I am sorry for the very late response.

It may be that the Travis is no longer running and thus the last changes are useless. But I hope it works again for this one pull request.

Thank you for your patience and help! 👍

@froschdesign
Copy link
Member

Already fixed with #8

Thank you for your time and this contribution anyway! 👍

@froschdesign froschdesign removed this from In progress in PHP 8.0 Apr 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request hacktoberfest-accepted Issues/Pull-Requests which can be fixed during Hacktoberfest: https://hacktoberfest.digitalocean.com
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PHP 8.0 support
4 participants