Differential D20027

Make the "PHP 7" setup warning more explicit about what it means
ClosedPublic
Actions

Authored by epriestley on Jan 23 2019, 10:36 PM.

Tags

None

Referenced Files

	F15532956: D20027.diff
	Wed, Apr 23, 5:44 PM

	F15507444: D20027.id47833.diff
	Tue, Apr 15, 7:30 PM

	F15502579: D20027.id47823.diff
	Mon, Apr 14, 2:00 AM

	F15498731: D20027.id.diff
	Sun, Apr 13, 2:04 PM

	F15492257: D20027.diff
	Sat, Apr 12, 12:49 PM

	F15426820: D20027.diff
	Mar 23 2025, 10:57 AM

	F15382256: D20027.id47833.diff
	Mar 14 2025, 11:31 AM

	F15368200: D20027.diff
	Mar 12 2025, 12:44 AM

View All Files

Subscribers

None

Details

Reviewers

amckinley

Commits

rP98c4cdc5bebf: Make the "PHP 7" setup warning more explicit about what it means

Summary

See https://discourse.phabricator-community.org/t/minimum-php-versions-supported-by-latest-stable-phabricator/2314/3.

Test Plan

o~o

Diff Detail

Repository

rP Phabricator

Branch

php7

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 21676
Build 29559: Run Core Tests
Build 29558: arc lint + arc unit

Event Timeline

epriestley created this revision.Jan 23 2019, 10:36 PM

Harbormaster completed remote builds in B21676: Diff 47823.Jan 23 2019, 10:38 PM

epriestley requested review of this revision.Jan 23 2019, 10:38 PM

TIL.

This revision is now accepted and ready to land.Jan 24 2019, 7:12 PM

Closed by commit rP98c4cdc5bebf: Make the "PHP 7" setup warning more explicit about what it means (authored by epriestley). · Explain WhyJan 24 2019, 11:09 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

config/

check/

PhabricatorPHPPreflightSetupCheck.php

19 lines

Diff 47823

View Options

src/applications/config/check/PhabricatorPHPPreflightSetupCheck.php

Make the "PHP 7" setup warning more explicit about what it meansClosedPublicActions