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.

Sign up for GitHub

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

Jump to bottom

dev/core#2204 Update minimum php install to be 7.2 #637

Merged
merged 1 commit into from
Jan 25, 2021
Merged

dev/core#2204 Update minimum php install to be 7.2 #637

merged 1 commit into from
Jan 25, 2021

Conversation

seamuslee001
Copy link
Contributor

as per civicrm/civicrm-core#19390 this updates the minimum install php version to be 7.2

@civibot
Copy link

civibot bot commented Jan 14, 2021

(Standard links)

@civibot civibot bot added the 7.x-5.34 label Jan 14, 2021
@seamuslee001 seamuslee001 mentioned this pull request Jan 14, 2021
Merged
@seamuslee001
Copy link
Contributor Author

Jenkins re test this please

@seamuslee001 seamuslee001 changed the base branch from 7.x-5.34 to 7.x-master January 17, 2021 20:40
@civibot civibot bot added 7.x-master and removed 7.x-5.34 labels Jan 17, 2021
@seamuslee001
Copy link
Contributor Author

Jenkins re test this please

@demeritcowboy
Copy link
Contributor

Tested. Looks good.
I note that cv core:check-req doesn't check the php version, but that's separate.

@seamuslee001
Copy link
Contributor Author

Merging as per @demeritcowboy review

@seamuslee001 seamuslee001 merged commit 524bbf3 into civicrm:7.x-master Jan 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
7.x-master
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@seamuslee001 @demeritcowboy