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

Bump PHP version #917

Closed
osma opened this issue Jan 31, 2020 · 0 comments · Fixed by #919
Closed

Bump PHP version #917

osma opened this issue Jan 31, 2020 · 0 comments · Fixed by #919
Assignees
Labels
enhancement php7 Tag for php7 features size-small max 2 hours tests
Milestone

Comments

@osma
Copy link
Member

osma commented Jan 31, 2020

Currently Skosmos requires 7.0 and supports versions 7.1 and 7.2. But PHP 7.3 and 7.4 have been available for some time. I suggest the following:

  • drop support for PHP 7.0 (this would enable using more advanced parameter type hints introduced in 7.1 - but actually doing that is out of scope for this issue)
  • try to run Skosmos under PHP 7.3 and 7.4
  • enable running Travis CI tests under 7.3 and 7.4 (with allowed_failures set to true if they cannot be run succesfully already)
  • open separate issues for any problems found with 7.3 and 7.4
@osma osma added enhancement tests php7 Tag for php7 features labels Jan 31, 2020
@osma osma added this to the Next Tasks milestone Jan 31, 2020
@osma osma added the size-small max 2 hours label Jan 31, 2020
@osma osma self-assigned this Jan 31, 2020
@kouralex kouralex modified the milestones: Next Tasks, 2.3 Feb 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement php7 Tag for php7 features size-small max 2 hours tests
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants