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 php 8.3 and remove php 8.0 as a dependency. #25

Open
wants to merge 7 commits into
base: 1.7.x
Choose a base branch
from

Conversation

datasage
Copy link

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

Description

Add PHP 8.3 as a supported version and removed php 8.0 as a supported version.

Included is an update to phpunit 10.x and psalm 5.x.

All the required packages were updated to the minimum version that matched the php version requirments

Signed-off-by: Dan Johansson <datasage@gmail.com>
@InstantlyMoist
Copy link

merge and release
merge and release
merge and release

src/ApiProblem.php Outdated Show resolved Hide resolved
src/ApiProblem.php Outdated Show resolved Hide resolved
src/ApiProblem.php Outdated Show resolved Hide resolved
Copy link
Member

@samsonasik samsonasik left a comment

Choose a reason for hiding this comment

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

Please revert ApiProblem change as bc break

Signed-off-by: Dan Johansson <datasage@gmail.com>
@datasage
Copy link
Author

@samsonasik Removed most of the type hints for now. Understand the concern about compatibility.

src/ApiProblem.php Outdated Show resolved Hide resolved
Signed-off-by: Dan Johansson <datasage@gmail.com>
Signed-off-by: Dan Johansson <datasage@gmail.com>
@kersysgediminas
Copy link

@samsonasik All requested changes were made, please review.

@svycka
Copy link
Contributor

svycka commented May 13, 2024

Hello @datasage, do you plan to update this PR? Do you need any help?

Signed-off-by: Dan Johansson <datasage@gmail.com>
@datasage
Copy link
Author

All CI checks are passing now

Signed-off-by: Dan Johansson <datasage@gmail.com>
@samsonasik
Copy link
Member

composer.lock need to be regenerated

Signed-off-by: Dan Johansson <datasage@gmail.com>
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.

None yet

5 participants