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 minimum version to 7.2 #49

Merged
merged 1 commit into from
Nov 16, 2024

Conversation

MauricioFauth
Copy link
Member

No description provided.

Signed-off-by: Maurício Meneghini Fauth <mauricio@mfauth.net>
@kamil-tekiela
Copy link
Contributor

Is this really needed? Why can it not stay as 7.1?

@williamdes
Copy link
Member

Is this really needed? Why can it not stay as 7.1?

Because tools are dropping the version too
and some of our libs already did drop the version
I would say

@MauricioFauth
Copy link
Member Author

Is this really needed? Why can it not stay as 7.1?

What William said and to stay in sync with phpMyAdmin 5.2 requirements. This should have been done when phpMyAdmin's PHP version was bumped to 7.2.

@MauricioFauth MauricioFauth merged commit 7865bec into phpmyadmin:5.4.x Nov 16, 2024
19 checks passed
@MauricioFauth MauricioFauth deleted the php7.2 branch November 16, 2024 16:34
@MauricioFauth MauricioFauth self-assigned this Nov 16, 2024
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.

3 participants